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

EMSUSD-1912 disable new light test on USD 24.11 #4033

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

pierrebai-adsk
Copy link
Collaborator

For some reasons, USD 24.11 reports the root layer as being modified even though dumping the layer as string shows the data is not modified. Will need to be investigated further.

For some reasons, USD 24.11 reports the root layer as being modified even though dumping the layer as string shows the data is not modified. Will need to be investigated further.
@pierrebai-adsk pierrebai-adsk added the bug Something isn't working label Dec 5, 2024
@pierrebai-adsk pierrebai-adsk self-assigned this Dec 5, 2024
Copy link
Collaborator

@seando-adsk seando-adsk left a comment

Choose a reason for hiding this comment

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

Okay. Just don't forget to remove the print statement when you fix the test and reenable it.

@seando-adsk seando-adsk added ready-for-merge Development process is finished, PR is ready for merge unit test Related to unit tests (both python or c++) and removed bug Something isn't working labels Dec 6, 2024
@seando-adsk seando-adsk merged commit 1193416 into dev Dec 6, 2024
11 checks passed
@seando-adsk seando-adsk deleted the bailp/EMSUSD-1912/light-test-problem-temp-fix branch December 6, 2024 12:21
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