William Parker
William Parker
+1 this could be useful. Some points for consideration: - Would we want a typical Java approach of having various classes represent the data returned or just Java maps say...
Is your need for this in Clojure or Clojurescript? My first thought for implementing this in Clojure would be to implement the RuleSource protocol. Something like ```` (defrecord NamespaceWithRulesBlacklist [ns...
My reason for suggesting having the exclusions and inclusions in a single ordered data structure was to allow for having well-defined priority between them. Otherwise expressing use-cases like "blacklist all...
OK fair enough on the issues surrounding ClojureScript eval. The API change probably doesn't add too much cognitive overhead for users as an option ":production-exclusions" is easily understood even without...
Regarding ```` For example: Does an accumulator have to be used to arrive the sort of problem behavior here? ```` I don't think so, but it makes it more consistent...
The :ns-name was added to rules to address https://github.com/cerner/clara-rules/issues/178 which concerned incorrect qualifying of symbols on the RHS of rules. Since queries don't have a RHS they weren't impacted. Similar...
@alex-dixon , is this having an impact on the work you referenced to reload rules dynamically or did you just happen to notice it in the course of that work?...
@alex-dixon I agree that the behavior you describe is undesirable. So as I understand this you're seeing this behavior in a REPL environment? I wonder if that cause is something...
@sparkofreason logged https://github.com/cerner/clara-rules/issues/359 (thanks for that by the way) for the issues discussed in his previous comment https://github.com/cerner/clara-rules/issues/327#issuecomment-347291708. Regarding the behavior of the map of rules stored in the ClojureScript...
+1 to @mrrodriguez 's diagnosis, and I agree that this looks like a bug. We've used Java wrappers around Clojure types to avoid this issue in the past; see [AlphaRootsWrapper...