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

Too many absolute paths #12

Open
boortel opened this issue Mar 23, 2021 · 0 comments
Open

Too many absolute paths #12

boortel opened this issue Mar 23, 2021 · 0 comments

Comments

@boortel
Copy link

boortel commented Mar 23, 2021

Hello all,

I tried to run the code in the repository, but too many paths are absolute and the only way how to fix it is to change them manually. I would like to try this code for my research, but this issue and also the not clear programming style makes it difficult to use.

I would like to suggest a major code clean up with one configuration file, adding the files for dataset (or at least links for them) and also to rewrite the code in order to make it less confusing (maybe to build up on the code of Lukas Ruff). A nice gadget would be a custom dataset loader template.

Is there a chance to make those changes, or did someone fix those issues? Thank you in advance!

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

1 participant