Skip to content

Conversation

@dj-mcg
Copy link
Collaborator

@dj-mcg dj-mcg commented Sep 13, 2024

Diffuse and Specular light parameter computations were adjusted, which affected several image-based tests. This introduces new baselines and USD version checking in these tests to ensure everything still passes

Diffuse and Specular light parameter computations were adjusted, which
affected several image-based tests. This introduces new baselines and
USD version checking in these tests to ensure everything still passes
@seando-adsk seando-adsk added the unit test Related to unit tests (both python or c++) label Sep 18, 2024
@dj-mcg
Copy link
Collaborator Author

dj-mcg commented Sep 19, 2024

Were there build or test failures here? I'm having trouble finding the logs, sorry!

@seando-adsk seando-adsk assigned dj-mcg and unassigned dj-mcg Sep 20, 2024
@seando-adsk
Copy link
Collaborator

@dj-mcg For security reasons we had to stop uploading the logs to github. I checked the internal build and there was a network glitch. I've restarted the preflight.

@seando-adsk seando-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Sep 20, 2024
@seando-adsk seando-adsk merged commit 7aec718 into Autodesk:dev Sep 20, 2024
3 of 4 checks passed
@dj-mcg dj-mcg deleted the pr/Update_Tests_For_UsdImaging_Changes branch September 20, 2024 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge Development process is finished, PR is ready for merge unit test Related to unit tests (both python or c++)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants