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

Houdini: fix default render product name in Vray #6083

Merged
merged 2 commits into from Mar 4, 2024

Conversation

antirotor
Copy link
Member

Changelog Description

This is fixing key name for default render products in VRay. Original name RGB Color caused issues during job submission.

Additional info

Fixing error like:

ayon_api.exceptions.GraphQlQueryFailed: GraphQl query Failed: Name 'render_ropMainRGB Color' does not match regex '^[a-zA-Z0-9]([a-zA-Z0-9.-]*[a-zA-Z0-9])?$' on item 'project/products' (Line 3 Column 5)

Testing notes:

  1. Run Houdini with Vray for Houdini
  2. Create VRay ROP and publish

@antirotor antirotor self-assigned this Dec 21, 2023
@ynbot ynbot added the size/XS Denotes a PR changes 0-99 lines, ignoring general files label Dec 21, 2023
@MustafaJafar
Copy link
Contributor

I don't have V-Ray to test it.
but changes in code matches the reported problem and suggested solution in discord.

Copy link
Member

@moonyuet moonyuet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested in Legacy OP looks good.
image
Tested in AYON, and it publishes successfully and render well.
image
image

@mkolar
Copy link
Member

mkolar commented Feb 7, 2024

@antirotor any reason this wasn't yet merged?

@MustafaJafar
Copy link
Contributor

This PR has been ported to Ayon ynput/ayon-core#117

@antirotor antirotor merged commit 57a0a58 into develop Mar 4, 2024
1 check passed
@antirotor antirotor deleted the bugfix/houdini-default-renderproduct-name branch March 4, 2024 08:55
@ynbot ynbot added this to the next-patch milestone Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
host: Houdini port to AYON size/XS Denotes a PR changes 0-99 lines, ignoring general files target: AYON target: OpenPype type: bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

5 participants