Skip to content
This repository was archived by the owner on Sep 20, 2024. It is now read-only.

Houdini: fix default render product name in Vray#6083

Merged
antirotor merged 2 commits intodevelopfrom
bugfix/houdini-default-renderproduct-name
Mar 4, 2024
Merged

Houdini: fix default render product name in Vray#6083
antirotor merged 2 commits intodevelopfrom
bugfix/houdini-default-renderproduct-name

Conversation

@antirotor
Copy link
Copy Markdown
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
Copy Markdown
Member

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

Copy link
Copy Markdown
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
Copy Markdown
Member

mkolar commented Feb 7, 2024

@antirotor any reason this wasn't yet merged?

@MustafaJafar
Copy link
Copy Markdown
Member

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

@antirotor antirotor merged commit 57a0a58 into develop Mar 4, 2024
@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 subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

5 participants