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 #213

Closed

Conversation

simonbethke
Copy link
Contributor

Here is an addition:

Examples:

comment Generated by Nerfstudio 1.0
comment Vertical Axis: z

comment Generated by SuperSplat 1.0
comment Vertical Axis: y

@simonbethke
Copy link
Contributor Author

This is basically just the first step that is required, to rectify issues like wrong up-axis during loading of files.
I filed basically the same PR for Nerfstudio and this would be the best answer for any issue anybody opens regarding "My file is opening with the wrong orientation, can you please fix it?"
And your answer should be: Contact the developer of your ply-producer and ask him to add the correct "comment Vertical Axis: "

@slimbuck
Copy link
Member

I'm not sure how useful this is going to be without adding support for all the possible Y-axis specifiers at load time. Won't the "Generated by" comments be enough for now?

@simonbethke
Copy link
Contributor Author

Is there any way, to implement the loading in this project?
I stumble upon this shortcome at every time loading ply files from nerfstudio and thus already got nerfstudio-project/nerfstudio#3472 merged

@slimbuck
Copy link
Member

Unfortuntely the engine's resource loader doesn't expose comments otherwise you could detect this quite easily :(

@slimbuck
Copy link
Member

We rolled out custom header entries before, but unfortunately it broke some user's workflow with external tools that don't support any variation on the the PLY header.

I think the best solution is to add optional custom header bits for users that want them.

For that though we'll need a save and load dialog. So I'm closing this for now till we're ready to add that.

@slimbuck slimbuck closed this Nov 11, 2024
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.

3 participants