Skip to content

BSPR-111077 - Speed up conversion of meshes with left-handed winding order.#4516

Merged
seando-adsk merged 1 commit intoAutodesk:devfrom
AntoineMilliezDisney:BSPR-111077
Mar 24, 2026
Merged

BSPR-111077 - Speed up conversion of meshes with left-handed winding order.#4516
seando-adsk merged 1 commit intoAutodesk:devfrom
AntoineMilliezDisney:BSPR-111077

Conversation

@AntoineMilliezDisney
Copy link

The original implementation made lots of calls to polygonVertexCount which could be slow (order of minutes) on large meshes.

@AntoineMilliezDisney AntoineMilliezDisney deleted the BSPR-111077 branch March 6, 2026 17:10
@AntoineMilliezDisney AntoineMilliezDisney restored the BSPR-111077 branch March 13, 2026 17:08
Copy link
Collaborator

@pierrebai-adsk pierrebai-adsk left a comment

Choose a reason for hiding this comment

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

You will need to fix the code layout according to teh clang failures. Click on the clang failure link to see the errors.

See: https://github.com/Autodesk/maya-usd/blob/dev/doc/codingGuidelines.md#c-coding-guidelines

@AntoineMilliezDisney
Copy link
Author

This builds and works fine at our company on Maya 2025 / Alma Linux, I see build failures in the build_preflight step which I cannot troubleshoot.
@pierrebai-adsk - Considering I am non-Autodesk and humbly trying to help here, are we okay having an in-house developer take it over?
Thanks a lot!

@pierrebai-adsk
Copy link
Collaborator

Don't worry about this failure, our internal build pipeline had hiccups. I'll restart the preflight once again.

@seando-adsk
Copy link
Collaborator

Build failure was an internal glitch. The preflight build passed. Marking as ready for merge.

@seando-adsk seando-adsk added import-export Related to Import and/or Export ready-for-merge Development process is finished, PR is ready for merge labels Mar 24, 2026
@seando-adsk seando-adsk merged commit 4319fba into Autodesk:dev Mar 24, 2026
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

import-export Related to Import and/or Export 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