Skip to content

Fix OSL definitions of matrix extract#2849

Merged
jstone-lucasfilm merged 1 commit intoAcademySoftwareFoundation:mainfrom
jstone-lucasfilm:dev_matrix_extract
Apr 1, 2026
Merged

Fix OSL definitions of matrix extract#2849
jstone-lucasfilm merged 1 commit intoAcademySoftwareFoundation:mainfrom
jstone-lucasfilm:dev_matrix_extract

Conversation

@jstone-lucasfilm
Copy link
Copy Markdown
Member

This changelist fixes a function overload error in the OSL definitions of the matrix extract nodes.

Fixes #2845.

This changelist fixes a function overload error in the OSL definitions of the matrix extract nodes.

Fixes AcademySoftwareFoundation#2845.
@jstone-lucasfilm
Copy link
Copy Markdown
Member Author

Thanks to @msuzuki-nvidia for catching this, and I'm CC'ing @JHUbisoft as an FYI.

Copy link
Copy Markdown
Contributor

@msuzuki-nvidia msuzuki-nvidia left a comment

Choose a reason for hiding this comment

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

LGTM!

@jstone-lucasfilm jstone-lucasfilm merged commit 2ae3121 into AcademySoftwareFoundation:main Apr 1, 2026
70 of 71 checks passed
@jstone-lucasfilm jstone-lucasfilm deleted the dev_matrix_extract branch April 1, 2026 20:11
@JHUbisoft
Copy link
Copy Markdown
Contributor

Thanks and sorry for the inconvenience

@jstone-lucasfilm
Copy link
Copy Markdown
Member Author

Not your fault, @JHUbisoft, and ultimately it's on us to integrate OSL validation into our GitHub CI, so that this category of error gets caught early.

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

Successfully merging this pull request may close these issues.

OSL reference generation test fails

3 participants