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

Unable to reproduce the results #7

Open
Arjung27 opened this issue Mar 8, 2021 · 2 comments
Open

Unable to reproduce the results #7

Arjung27 opened this issue Mar 8, 2021 · 2 comments

Comments

@Arjung27
Copy link

Arjung27 commented Mar 8, 2021

Hi! I downloaded the data using the link mentioned in the readme (13G almost) and tried running the code. I could not reproduce the numbers. Rather my numbers are way off. The training is highly unstable and the loss value is very noisy (not going down rather oscillating). I also tried adding RMVSNet to reproduce the numbers on that but wasn't able to do so. I think there is some issue with the data or data loader for WHU dataset. Could you or anyone please verify if you or they are able to reproduce the numbers using the repo?

@gpcv-liujin
Copy link
Owner

Hi! I downloaded the data using the link mentioned in the readme (13G almost) and tried running the code. I could not reproduce the numbers. Rather my numbers are way off. The training is highly unstable and the loss value is very noisy (not going down rather oscillating). I also tried adding RMVSNet to reproduce the numbers on that but wasn't able to do so. I think there is some issue with the data or data loader for WHU dataset. Could you or anyone please verify if you or they are able to reproduce the numbers using the repo?

Sorry that we updated the camera parameters in the training data, but forgot to update the code at the same time.
You need to replace the following code in " def tr_load_cam( )" of "preprocess.py" file :

x0 = pera[0][1] - pera[0][7] - (pera[0][10] * pera[0][11])  # whu
y0 = pera[0][2] - pera[0][8] - (pera[0][9] * pera[0][12])

To:

x0 = pera[0][1] # whu
y0 = pera[0][2]

The new version of the code has been re-uploaded.
Thank you very much for your feedback, and sorry again for causing trouble to you.

@Arjung27
Copy link
Author

Arjung27 commented Mar 9, 2021

Thank you for clarifying. I will make that change. Give me a day to train the model and I will get back to you asap.

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