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

misleading contents #54

Closed
AIBluefisher opened this issue Jan 19, 2023 · 1 comment
Closed

misleading contents #54

AIBluefisher opened this issue Jan 19, 2023 · 1 comment

Comments

@AIBluefisher
Copy link

I think the major part of this project is only a copy of Mega-NeRF. While in the code and README, this repo uses BlockNeRF and claims it supports BlockNeRF, which is actually not true and can be misleading to others. I think this issue should be fixed.

@sjtuytc
Copy link
Owner

sjtuytc commented Jan 19, 2023

Thanks for your attention and suggestions.

  1. The Mega-NeRF has been removed. I'm busy with the next major release, so the docs seem to be inconsistent temporally. I don't think our repo will be related to Mega-NeRF in the future (because the setup and backbones are quite different).
  2. As for the improved Block-NeRF, it is explained in No such file or directory train_block_nerf.py #45 . I have tried my best to make it online and I have been working on it for months. If you have a quicker or more accurate implementation, you can start and contribute to a new GitHub repo or create a PR to this repo.

@sjtuytc sjtuytc closed this as completed Jan 19, 2023
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

No branches or pull requests

2 participants