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-252 relative payload preview #3229

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

pierrebai-adsk
Copy link
Collaborator

Fix the preview of the relative path for payloads, references and assets (textures).

  • Rename the field _parentLayerPath to _relativeToDir since it is not always a parent layer folder.
  • Add explicit flag to use the scene file path as the relative base.
  • Otherwise, the scene file would be used when the layer is not yet saved, which would be incorrect and misleading.
  • Set _relativeToDir in the usdFileRelativeToEditTargetLayer uiInit function.
  • Failing to do that made the base class use the scene file folder instead, which was incorrect.

Fix the preview of the relative path for payloads, references and assets (textures).

- Rename the field _parentLayerPath to _relativeToDir since it is not always a parent layer folder.
- Add explicit flag to use the scene file path as the relative base.
- Otherwise, the scene file would be used when the layer is not yet saved, which would be incorrect and misleading.
- Set _relativeToDir in the usdFileRelativeToEditTargetLayer uiInit function.
- Failing to do that made the base class use the scene file folder instead, which was incorrect.
@pierrebai-adsk pierrebai-adsk added bug Something isn't working adsk Related to Autodesk plugin labels Jul 14, 2023
@pierrebai-adsk
Copy link
Collaborator Author

The only PF failure is one of the known random export failures when renaming temp files.

@pierrebai-adsk pierrebai-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Jul 14, 2023
@seando-adsk seando-adsk merged commit df33b99 into dev Jul 17, 2023
@seando-adsk seando-adsk deleted the bailp/EMSUSD-252/relative-payload-preview branch July 17, 2023 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adsk Related to Autodesk plugin bug Something isn't working ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants