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

Unable to use multiple constraints as only first is selected #7

Open
rhysied opened this issue Mar 20, 2023 · 0 comments
Open

Unable to use multiple constraints as only first is selected #7

rhysied opened this issue Mar 20, 2023 · 0 comments

Comments

@rhysied
Copy link

rhysied commented Mar 20, 2023

Hi,

I have been using the v2 ArtEngine recently and discovered an issue with a fairly complex setup that requires multiple edge-cases (in our case we have a pose that inherits from separate skin and armour palettes).

The issue appears to be related to this break; in the ImageLayersAttributesGenerator which stops the loop from continuing and being able to find 2nd/Nth matching constraints.

I commented out the value in my local version to test and have confirmed that removing it means both of our edge-cases are matched, however this is less efficient as its potentially looping through unnecessarily for items that only have single edge-cases, so I'm not sure whether you want to accept the performance hit of the unnecessary looping or try and refactor it.

Cheers, Rhys.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant