Skip to content

Conversation

SergioRZMasson
Copy link
Contributor

@SergioRZMasson SergioRZMasson commented Apr 24, 2025

When serializing to .babylon file we are setting the _isExpanded property directly in the array object to flag if matrices indices are in compact or expanded form. However, JSON.stringify will remove those properties since it does not support properties in array objects. I'm moving those flags to the serializationObject itself to allows it to be properly exported. I'm also keeping the deserializer for it just in case someone has a file that somehow has that property in the array.

Releated to Exporters PR: BabylonJS/Exporters#1137

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 24, 2025

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 24, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 24, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 24, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 24, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 24, 2025

@deltakosh deltakosh enabled auto-merge (squash) April 28, 2025 14:40
@bjsplat
Copy link
Collaborator

bjsplat commented Apr 28, 2025

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 28, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 28, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 28, 2025

@deltakosh deltakosh merged commit 9700f01 into BabylonJS:master Apr 28, 2025
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants