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

Test cameras in ZJU differ from the description in the papers #65

Open
ChenYutongTHU opened this issue Jan 23, 2024 · 4 comments
Open

Comments

@ChenYutongTHU
Copy link

ChenYutongTHU commented Jan 23, 2024

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]

cfg.test_view = []

if len(cfg.test_view) == 0:
test_view = [
i for i in range(num_cams) if i not in cfg.training_view
]
if len(test_view) == 0:
test_view = [0]
else:
test_view = cfg.test_view

Thank you!

@ChenYutongTHU
Copy link
Author

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.
Another distinction from the paper is that for subject 377, the WarmUp sequence, in its config.yaml, cfg.num_eval_frame=200.

num_train_frame: 300
num_eval_frame: 200

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!

@pengsida
Copy link
Member

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.

@ChenYutongTHU
Copy link
Author

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?

@pengsida
Copy link
Member

Please see the config file. I use four cameras for training and remaining cameras for test.

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