-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ParameterFilter.GetRule Node #190
Comments
can i have the graph file? (add ".txt" to the extension to upload) |
Hi Thanks for the reply. I had not looked at those samples and I appreciate the nudge. However I am still having issue getting a specific set of rules. I have included a .RVT and a .DYN to replicate the below image results. Multiple leveled rules are causing this error in the GetRule node: Unless i'm using a poor workflow, I'm trying to get all associated rules prior to removing the rules, rebuilding a ruletree and adding it back to the parameterFilter, because i cannot find a node that will simply add a rule into an existing rule tree. I can add a new rule tree but can't seem to get the rule to slot into an existign tree. Thanks very much for taking a look |
Amazing! |
aha... I have not coded it for that purpose... I will try to see what can be done :-) |
well adding and replacing should be doable today... if you see the sample I referred to earlier... in upper left side of the graph, I show how to work with the "ruletree", which will be the "filtertree" from hereon. However, deleting in a tree seems to be missed out now when i look at the sample :-) |
I have now worked a bit with Filters, and along the way, I started to remember why I made it as I did. The key is the ”Tree” since you can't edit a single rule or logical thing and then push it back. You need to ”create” what is needed and then add or replace this in the tree structure, or by removing what is needed. Hereafter you can bind it to a parameterFilter. In other words, it makes no sense to retrieve the individual thing in the tree. The tree is only needed to ”point” towards the change… either by adding, replacing, or removing. However, I notice, that I could make it better or more ”explaining” when showing the tree structure, so I will work a bit on that. Besides that, I will create a ”replace” and ”remove” node… From what you have send me I do think that my sample collection show you how you can use the nodes building the ”trees” you need, however, I will start with the replace/remove nodes and work on a better tree as the last thing. |
I have now released the new nodes and a new ruletree node. |
G'day, yeah I understand what you're saying. Thanks so much, I appreciate the updates, and the new nodes are working great for me so far. |
Operating system
Windows 10
Revit and Dynamo version
Revit 2023.1
DynamoRevit: 2.16.1.6510
DynamoCore: 2.16.1.2727
Orchid: 213.2.0.8342
What did you do?
Connected Parameter Filters node to ParameterFilter.GetRule node
What did you expect to see?
Expected to get list of Rules
What did you see instead?
Warning/Error per screen shot:
For reference the Rules have both And and Or per below screen shot:
Not sure if it helps you but in Revit Lookup selecting the Filter leads to a list with both an Element Parameter Filter and a Logical Or Filter:
It seems the logical Or filter is tripping up the "GetRule" node
The text was updated successfully, but these errors were encountered: