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

SSCBench-KITTI360 test results #24

Closed
joonsu0109gh opened this issue Jul 12, 2024 · 9 comments
Closed

SSCBench-KITTI360 test results #24

joonsu0109gh opened this issue Jul 12, 2024 · 9 comments
Assignees

Comments

@joonsu0109gh
Copy link

joonsu0109gh commented Jul 12, 2024

Hello, Thank you for the great work.

I have a question regarding the performance on the KITTI-360 dataset. Are the numbers you provided from the test set correct?

Based on the logs you shared, it seems that the results are from the validation set. In my experiments using your model weights, I obtained 'val IoU: 0.1835' and 'test IoU: 0.1782.' Could you please clarify this?

@npurson
Copy link
Member

npurson commented Jul 14, 2024

Thank you for bringing this to our attention!

To clarify, I have modified my local code to evaluate the test set as the validation set for convenience. As a result, the validation metrics reported in the log are actually the test metrics. However, I noticed that the dataset has been updated since we conducted our experiments [1], so I'm unsure if this is causing the discrepancy.

We will further investigate this issue, but unfortunately, it may take some time due to the need to re-prepare the entire dataset and our current workload. If you are urgent to report the results for comparison, you can first report your reproduced result of our method temporarily.

@npurson npurson self-assigned this Jul 14, 2024
@npurson
Copy link
Member

npurson commented Jul 16, 2024

Hi, one more thing to confirm: have you adapted the disparity value for KITTI-360 as discussed here?

@joonsu0109gh
Copy link
Author

joonsu0109gh commented Jul 16, 2024 via email

@g-ch
Copy link

g-ch commented Jul 25, 2024

Hi all,

@npurson @joonsu0109gh Could you share the KITTI-360 depth images generated with with pre-trained MobileStereoNet described in README? I tried to run the script but my hardware is not so good and it had memory overflowing issue.

Thank you in advance!

@npurson
Copy link
Member

npurson commented Jul 26, 2024

Hi all,

@npurson @joonsu0109gh Could you share the KITTI-360 depth images generated with with pre-trained MobileStereoNet described in README? I tried to run the script but my hardware is not so good and it had memory overflowing issue.

Thank you in advance!

I'm afraid that the depth images are too large (~ 10G) to upload. Also, it's worth noting that the inference of MobileStereoNet shouldn't consume a lot of memory—it's much less than what our network requires. Perhaps there might be some problems with your running scripts?

@g-ch
Copy link

g-ch commented Jul 28, 2024

Hi all,
@npurson @joonsu0109gh Could you share the KITTI-360 depth images generated with with pre-trained MobileStereoNet described in README? I tried to run the script but my hardware is not so good and it had memory overflowing issue.
Thank you in advance!

I'm afraid that the depth images are too large (~ 10G) to upload. Also, it's worth noting that the inference of MobileStereoNet shouldn't consume a lot of memory—it's much less than what our network requires. Perhaps there might be some problems with your running scripts?

Hi @npurson ,

Thank you so much for the reply! You are right. It's because my GPU is relatively new (3080) and the environment yaml file given by MobileStereoNet uses low version CUDA and pytorch. I rebuild the environment and upgrade to new versions and it works.

@Samsara011
Copy link

@npurson
Hello author, thank you for your work, please ask how much video memory is needed for the entire model training!
thanks!

@npurson
Copy link
Member

npurson commented Oct 24, 2024

@npurson Hello author, thank you for your work, please ask how much video memory is needed for the entire model training! thanks!

If you're referring to GPU memory, it's ~18GB.

@Samsara011
Copy link

@npurson
I see,thanks!

@npurson npurson closed this as completed Dec 11, 2024
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

4 participants