-
Notifications
You must be signed in to change notification settings - Fork 52
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
Test cameras in ZJU differ from the description in the papers #65
Comments
In fact, I found for subject 313, there are only 21 cameras. Camera (20)(21) are missing. For subjects 387 and 377, there are 23 cameras. animatable_nerf/configs/aninerf_377.yaml Lines 28 to 29 in f9b29c5
However, in the paper's table, the number of test frames is 317. Which test set did you use to report the number in the table? Great thank! |
In the TPAMI version, we use this config: https://github.com/zju3dv/animatable_nerf/blob/master/configs/aligned_nerf_pdf/aligned_aninerf_pdf_377.yaml#L28, which will test all the remaining frames with an interval, resulting in 317 test frames. |
I see. So, there are 617 frames in total, (I checked the dataset). I guess with 300 frames for training, there are 317 remaining frames for testing, which are then sampled with interval=30. Then how about the number of cameras? |
Please see the config file. I use four cameras for training and remaining cameras for test. |
Hi,
The paper mentions that 'using a multi-camera system that has 21 synchronized cameras. We select four cameras for training and use the remaining cameras for testing.'
However, I checked the config.yaml. For ZJU dataset, cfg.training_view is set [0,6,12,18] And there are 23 cameras in total. So the remaining views for testing are [1,2,3,4,5,7,8,9,10,11,13,14,15,16,17,19,20,21,22]
animatable_nerf/lib/config/config.py
Line 24 in f9b29c5
animatable_nerf/lib/datasets/tpose_pdf_dataset.py
Lines 26 to 33 in f9b29c5
Thank you!
The text was updated successfully, but these errors were encountered: