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

added center-method focus to sdfstudio-data #196

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

marcostrinca
Copy link

A quick update do sdfstudio dataparser allowing users to choose "focus" to center poses.
An example: ns-train neus sdfstudio-data ./mydata --center-method focus
Tested with neus and neus-facto with my own datasets after running process_nerfstudio_to_sdfstudio.py

I've used the code from the updated version of nerfstudio repo.

Note: the change I did in nerfstudio_collate.py to remove torch._six.string_classes was to make it compatible with new versions of pytorch. I've been using pytorch 2.0.1 by the way.

@marcostrinca
Copy link
Author

After reading #90 I thought this PR was unnecessary. Took me a couple minutes to realize the first is related to nerfstudio dataparser instead of sdfstudio dataparser.
Sorry about the mess!

@marcostrinca marcostrinca reopened this Aug 12, 2023
@niujinshuchong
Copy link
Member

@marcostrinca Thanks for the PR but seems like this PR contains multiple changes (centering, adding mini-omidata, foreground masks). Would you mind split it to different PRs?

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