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

dynamic KITTI sequence mentioned in your paper seems to be inconsistent. #18

Closed
WYKXD opened this issue Jan 23, 2024 · 6 comments
Closed

Comments

@WYKXD
Copy link

WYKXD commented Jan 23, 2024

Hello, after visualizing the dynamic KITTI download link you gave, the qualitative result picture of the dynamic KITTI sequence mentioned in your paper seems to be inconsistent. Did you use other sequences when testing?

@SecureSheII
Copy link
Member

Dynamic KITTI is a strict subset of KITTI so all images you find in Dynamic KITTI are copied from KITTI. The image quality should be the same as KITTI as well as images shown in the paper. May I ask what "qualitative result picture" are you referring to?

@WYKXD
Copy link
Author

WYKXD commented Jan 23, 2024 via email

@WYKXD
Copy link
Author

WYKXD commented Jan 23, 2024

Are the KITTI evaluation results in your paper obtained from the KITTI dynamic sequences you gave? I used the code and weights you published to evaluate and got inconsistent results, especially on the sequence of 04_0, it is best to get only 0.85

@SecureSheII
Copy link
Member

Have you used the correct model weights for KITTI? On 04_0 which has 36 frames in total, the ATE is 0.0180 using the finetuned model and 0.0206 if loading the flow model and pose model separately. Regarding sequence 01, we report the result of 01_1 in our paper, the one with 133 frames in total. For the other sequences, we use 00_1 (28 frames), 01_1 (133 frames), 02_1 (67 frames), 03_0 (31 frames), 04_0 (36 frames), 07_1 (136 frames), 08_0 (40 frames), 10_1 (59 frames).

If you still get inconsistent results, please provide instructions on how to reproduce the result you got.

@WYKXD
Copy link
Author

WYKXD commented Jan 29, 2024

Thank you very much

@SecureSheII
Copy link
Member

Thanks to @Xie-HIT, the reason might be found, which is on our end. Please refer to #20 (comment)

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