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

V2 #62

Open
wants to merge 9 commits into
base: v2
Choose a base branch
from
Open

V2 #62

wants to merge 9 commits into from

Conversation

CurtinComputing
Copy link

Hi developers,

I find there are still a lot of issues caused by the code bugs on the Nerfies@v2.
These issues has been opened on the issue sections inclduing #58 #59 #53 #44 .

The main correct on the bugs of code is as listed as below:

  1. Under coding file "configs.py", at line 19 "from flax import nn" should be corrected to "from flax import linen as nn"
  2. Under coding file "model_utils.py", at line 107 " jnp.broadcast_to([last_sample_z], z_vals[..., :1].shape)" should be corrected to " jnp.broadcast_to(jnp.array([last_sample_z]), z_vals[..., :1].shape)"

After fixing these bugs of the codes, nerfies can run the with the prepaired custmised image data.

Larry

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

1 participant