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

tag exported ply files #3472

Merged
merged 4 commits into from
Oct 14, 2024

Conversation

simonbethke
Copy link
Contributor

@simonbethke simonbethke commented Oct 11, 2024

I would like to write the keyword "Nerfstudio" into exported ply files so other software can compensate for the 90° rotation Nerfstudio applies to the data (it doesn't really matter what is the standard and what isn't. It is generally helpful, to have the possibility to know who wrote a file)

@brentyi
Copy link
Collaborator

brentyi commented Oct 11, 2024

Would it make sense to include the nerfstudio version?

I think it can be read via:

from importlib.metadata import version
nerfstudio_version = version("nerfstudio")

@simonbethke
Copy link
Contributor Author

That would totally make sense. I was not aware of any way to read it :)

@simonbethke
Copy link
Contributor Author

Examples:

comment Generated by Nerfstudio 1.0
comment Vertical Axis: z

comment Generated by SuperSplat 1.0
comment Vertical Axis: y

Copy link
Collaborator

@brentyi brentyi left a comment

Choose a reason for hiding this comment

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

Looks great, thanks @simonbethke!

@brentyi brentyi merged commit adc743c into nerfstudio-project:main Oct 14, 2024
3 checks passed
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