Skip to content

Accept extended splats when writing SPZ - #440

Open
oscarlorentzon wants to merge 2 commits into
sparkjsdev:mainfrom
oscarlorentzon:spz-write-extsplats
Open

oscarlorentzon wants to merge 2 commits into
sparkjsdev:mainfrom
oscarlorentzon:spz-write-extsplats

Conversation

@oscarlorentzon

Copy link
Copy Markdown
Collaborator

writeSpz accepted only PackedSplats, so an ExtSplats could not be written. ExtSplats is now accepted too, and its positions reach the encoder as the 32 bit floats it stores rather than the half floats PackedSplats holds. Calls that pass a PackedSplats still work. The types describing what transcodeSpz accepts are now exported, so an application can import them instead of restating the shape.

The loader stopped using these types when the function taking them moved
away. The argument type is now exported too, so callers can name what
they pass.
Only the packed container was accepted, so extended splats had to be
converted first, rounding their positions.

@asundqui asundqui left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

So fast! Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants