ns-render - render all images - #2459
Conversation
|
Now, I am not sure if this is needed. It seems like the |
|
Please let me know what you think... |
|
I'm not familiar with ns-eval, I think @FrederikWarburg made it. It does seem cleaner to have just one ns-render command which supports an 'eval' mode rather than an ns-eval script though. Fred do you have any thoughts? |
|
Hi, I don't know about ns-eval. I agree that it does not seem super useful. I think what we did in nerfbuster is the easiest way to evaluate. Here we render out images using ns-render and save these renderings on disk (you can do this with either training or evaluation images. When you train your nerf you can decide how to split the train/eval set), and then we have a separate script that computes metrics. I believe ns-eval tries to do this without saving the renderings to disk. |
|
Yes I agree. I was missing the functionality to render all train/eval images. |
|
@jkulhanek another issue with ns-render is that it doesn't use the full set of intrinsics (unless it was fixed since #2059). Does this use all intrinsics, including the image size? Images may have different sizes. With #2059, the "dataset" render mode is not necessary: I use Sorry I didn't have time to provide sample outputs for #2059 so it was never merged, but could you take a look at it? |
|
@f-dy
Overall I think it's nice to have both command |
|
I was using |
|
Can we merge this? |
ns-render: add support for rendering of all images.