Skip to content
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

Add FilterUnion node to combine the results of two or more filters #594

Closed
bentoogood opened this issue Oct 12, 2013 · 1 comment · Fixed by #652
Closed

Add FilterUnion node to combine the results of two or more filters #594

bentoogood opened this issue Oct 12, 2013 · 1 comment · Fixed by #652
Labels
scene Issues with GafferScene

Comments

@bentoogood
Copy link
Contributor

An example usage would be:

PathFilterA returns the paths to match charAssetA from a scene cache.
PathFilterB returns the paths to match charAssetB from a scene cache.

user wants to apply one operation to both characters. her current options are to:

  • make another filter can duplicate the expressions/paths from PathFilterA & B
  • use two duplicate nodes to apply the same operation (using one of the two original filters for each of the two nodes)

a workflow that might be preferable is to use a FilterUnion to combine the results of the two orig filters into one uber filter.

@johnhaddon
Copy link
Member

We should also give some thought to a general purpose BooleanFilter - the thinking being whether or not we can support the AncestorMatch and DescendantMatch rules properly.

johnhaddon added a commit to johnhaddon/gaffer that referenced this issue Nov 5, 2013
Also moved the bindings for all filters into a separate file to further improve GafferSceneModule.cpp compilation times.

Fixes GafferHQ#594.
johnhaddon added a commit to johnhaddon/gaffer that referenced this issue Nov 5, 2013
Also moved the bindings for all filters into a separate file to further improve GafferSceneModule.cpp compilation times.

Fixes GafferHQ#594.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scene Issues with GafferScene
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants