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

Custom phototourism data loader #72

Open
wants to merge 3 commits into
base: nerfw
Choose a base branch
from

Conversation

Dawars
Copy link

@Dawars Dawars commented Apr 25, 2021

Added script to generate TSV file from colmap reconstruction (undistort needed).

Dataloader properly loads data when camera id != image id

@kwea123
Copy link
Owner

kwea123 commented Apr 26, 2021

The code looks good, can you tell me which scene you're using, so that I can verify this works?

@Dawars
Copy link
Author

Dawars commented Apr 26, 2021

I used my own colmap reconstruction available here:
https://drive.google.com/file/d/1dgfQIomKb2AAV5asaWhRcgtyl--ABsgW/view?usp=sharing

@kwea123
Copy link
Owner

kwea123 commented Apr 26, 2021

Thanks, have you successfully trained it? And do you have a working checkpoint?

@Dawars
Copy link
Author

Dawars commented Apr 26, 2021

I did, here it is epoch=19.ckpt.zip

061

@Dawars
Copy link
Author

Dawars commented Oct 21, 2023

Is there anything against merging this?

I'm happy to make the necessary changes

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