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

Add NeRF-- #29

Merged
merged 1 commit into from Feb 16, 2021
Merged

Add NeRF-- #29

merged 1 commit into from Feb 16, 2021

Conversation

mwtarnowski
Copy link
Contributor

NeRF--: Neural Radiance Fields Without Known Camera Parameters
by Zirui Wang, Shangzhe Wu, Weidi Xie, Min Chen, Victor Adrian Prisacariu.

An end-to-end framework for training NeRF models given only RGB images, without pre-computed camera parameters.

@kwea123
Copy link
Contributor

kwea123 commented Feb 16, 2021

As I observed from a lot of experiments I have conducted with NeRF (on real world car scenes), it is true that NeRF doesn't require any "correct" intrinsics+extrinsics to achieve good results. By "correct" I mean careful camera calibration using checkerboard and so on. Rather, using these correct values, surprisingly, produces inferior performance. Hence now I'm always using colmap estimated values even if they are not "correct", and it also seems that there exists a lot of possible intrinsics+extrinsics combinations that make NeRF work well.

This work confirms my experience that many calibrations work for NeRF, and it is possible to learn one of them. I'm looking forward to experimenting how it performs with my data and how much longer it takes compared to NeRF+COLMAP (there are more parameters to optimize, so I expect a longer training time, but at the same time it saves COLMAP time).

@yenchenlin
Copy link
Collaborator

Hello @mwtarnowski , thanks again!

Same as the previous PR, I have put the citation of this paepr here. Could you update your PR to cite this txt file?

Very helpful and timely PR!

@yenchenlin
Copy link
Collaborator

@kwea123 Why not colmap init + updating camera parameters during NeRF optimization?

@mwtarnowski
Copy link
Contributor Author

Done.

@yenchenlin yenchenlin merged commit 4baf982 into awesome-NeRF:main Feb 16, 2021
@yenchenlin
Copy link
Collaborator

Thank you!

@mwtarnowski mwtarnowski deleted the add-nerfmm branch February 16, 2021 12:42
@kwea123
Copy link
Contributor

kwea123 commented Feb 16, 2021

@kwea123 Why not colmap init + updating camera parameters during NeRF optimization?

Yes, it also improves the score, as shown by the paper. It's something I'd like to implement and test in the future.

@yenchenlin btw, I have updated my progress on NeRF-W #24 and #25, please review it when you have time, thanks.

@yenchenlin
Copy link
Collaborator

@kwea123 no problem!

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

3 participants