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

Fix OpenGL issue in Python virtual environment #1766

Merged

Conversation

howetuft
Copy link
Contributor

@howetuft howetuft commented Apr 5, 2024

See issue #1755

Copy link

linux-foundation-easycla bot commented Apr 5, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link
Member

@jstone-lucasfilm jstone-lucasfilm left a comment

Choose a reason for hiding this comment

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

This proposed change looks good to me, thanks @howetuft, and ideally we should include it in the upcoming 1.38.10 release.

@howetuft
Copy link
Contributor Author

Thank you very much. I close!

@howetuft howetuft closed this Apr 11, 2024
@howetuft howetuft reopened this Apr 11, 2024
@howetuft
Copy link
Contributor Author

Sorry, I misunderstood (thought you had merged!!!)

@jstone-lucasfilm jstone-lucasfilm merged commit 091c35f into AcademySoftwareFoundation:main Apr 11, 2024
61 checks passed
kwokcb added a commit to kwokcb/MaterialX that referenced this pull request Apr 19, 2024
* Fix OpenGL issue in Python virtual environment (AcademySoftwareFoundation#1766)

See issue AcademySoftwareFoundation#1755

* Finalize changelog for 1.38.10

* Improvements to GitHub CI

- Upgrade the MacOS_Xcode_14_Python311 build to the MacOS 14 environment.
- Remove unneeded steps in virtual framebuffer setup and JavaScript uploads.

* Remove unused classes

This changelist removes two unused classes, SurfaceShaderNodeGlsl and SurfaceShaderNodeMsl, which are not leveraged in any active code path for MaterialX shader generation.

* Add ESSL unit tests

This changelist adds initial unit tests for ESSL shader generation, bringing the measured code coverage of MaterialX unit tests to 88.7%.

---------

Co-authored-by: howetuft <34142192+howetuft@users.noreply.github.com>
Co-authored-by: Jonathan Stone <jstone@lucasfilm.com>
kwokcb added a commit to kwokcb/MaterialX that referenced this pull request Apr 19, 2024
* Fix OpenGL issue in Python virtual environment (AcademySoftwareFoundation#1766)

See issue AcademySoftwareFoundation#1755

* Finalize changelog for 1.38.10

* Improvements to GitHub CI

- Upgrade the MacOS_Xcode_14_Python311 build to the MacOS 14 environment.
- Remove unneeded steps in virtual framebuffer setup and JavaScript uploads.

* Remove unused classes

This changelist removes two unused classes, SurfaceShaderNodeGlsl and SurfaceShaderNodeMsl, which are not leveraged in any active code path for MaterialX shader generation.

* Add ESSL unit tests

This changelist adds initial unit tests for ESSL shader generation, bringing the measured code coverage of MaterialX unit tests to 88.7%.

---------

Co-authored-by: howetuft <34142192+howetuft@users.noreply.github.com>
Co-authored-by: Jonathan Stone <jstone@lucasfilm.com>
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.

2 participants