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

Update pose_utils.py #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

starhiking
Copy link

fix the error when existing invalid views in the scene.
Beside the pose_boundary.npy will be generated, it also will generate "view_imgs.txt" to mark which views are used.

fix the error when existing invalid views in the scene.
Beside the pose_boundary.npy will be generated, it also will generate "view_imgs.txt" to mark which views are used.
@Sazoji
Copy link

Sazoji commented Apr 2, 2022

You deserve a metal for this, edited the model loading to handle the tuple for a separate colmap2poses file after I got fed up with total rejection when trying to map video just to get stuck with img2poses failing at the end while the colmap2nerf spaghetti passed totally fine.

@Achint08
Copy link

Achint08 commented Jun 2, 2022

Saviour!

@RiverOtherSide
Copy link

thanks!!!

@Holmes-Alan
Copy link

Thanks!!

@lagrange10
Copy link

thanks!! I am new in GitHub, only the third time I saw this issue did I see the awesome solve.

@cezarbbb
Copy link

cezarbbb commented Aug 8, 2023

Awesome 😎 bro! Appreciate that.

@BingyuJi
Copy link

oh~ you are so 🐂 Thanks

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.

None yet

9 participants