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

Maya: Fix Validate Attributes plugin #4401

Merged
merged 1 commit into from
Feb 1, 2023

Conversation

BigRoy
Copy link
Collaborator

@BigRoy BigRoy commented Jan 31, 2023

Brief description

Code was broken. So either plug-in was unused or it had gone unnoticed.

Description

Looking at the commit history of the plug-in itself it seems this might have been broken somewhere between two to three years. I think it's broken since two years since this commit.

Should this plug-in be removed completely?
@tokejepsen Is there still a use case where we should have this plug-in? (You created the original one)

Testing notes:

  1. Validate Attributes in maya, yay!

@tokejepsen
Copy link
Member

What is the alternative to validating attributes in Maya if we remove this?
We have been using it for production pipeline which is still on Pype 2.0, so around 2 years old seems about right.

@BigRoy
Copy link
Collaborator Author

BigRoy commented Jan 31, 2023

What is the alternative to validating attributes in Maya if we remove this?

As far as I know there's no generic implementation which checks whether a node name exists and check its attribute. For rendering there are certain validators that check node type and their attributes, but only really for rendering per specific render engine instead of something as generic as this.

So if there's need for this - then let's keep the fixed version. It does mean however, we should get some automated tests in for this one so that ha! It doesn't stay broken for another two years?

@tokejepsen
Copy link
Member

So if there's need for this - then let's keep the fixed version. It does mean however, we should get some automated tests in for this one so that ha! It doesn't stay broken for another two years?

Or we (Bumpybox) just need to hike up our boots and get the pipeline updated :)

Thanks for the fix though, we'll test it in 2 years time (Jokes! Hopefully...)

@BigRoy BigRoy added type: bug Something isn't working host: Maya labels Jan 31, 2023
@iLLiCiTiT iLLiCiTiT merged commit 185a7be into ynput:develop Feb 1, 2023
@github-actions github-actions bot added this to the next-patch milestone Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
host: Maya type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants