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.

Sign up for GitHub

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

Jump to bottom

Fix Dataset and Config Issues #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix Dataset and Config Issues #21

wants to merge 2 commits into from

Conversation

pbonazzi
Copy link

@pbonazzi pbonazzi commented Apr 1, 2024 •
edited
Loading

Hi,

There were some dependencies issues and some old code which hasn't been pushed blocking the script.

CUDA_VISIBLE_DEVICES=0 python train.py --config configs/train_e2depth_si_grad_loss_statenet_ergb.json

This was referenced in #1 .

And is fixed in 198a9a011e4f12f10f47fb3da6c59f365f0c2fa6

Best, Pietro

@pbonazzi
Copy link
Author

pbonazzi commented Apr 1, 2024

Hi,

I noticed many struggled to prepare the dataset (around 6 issues on the topic).
#10 #22 #20 #11 #2 #7

In commit 5aee9664696efaebb86194fb178ac0645b9fa016. I added a download_dataset.sh and refactored some of the variables which were found problematic.

@pbonazzi pbonazzi changed the title Fixed Dependencies Issues Fix Dataset and Config Issues Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pbonazzi