-
Notifications
You must be signed in to change notification settings - Fork 321
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
Add tum facade dataset #613
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @ThomasFroech this PR modifies the API for update_probs()
. Can you explain the reasoning behind that?
Hi @ThomasFroech can you add a readme.md for the dataset with this information: Website: |
Hi @ssheorey,
There is a README.md available in the original GitHub repo (https://github.com/OloOcki/tum-facade). I think that all important information is there. While the data can be downloaded from that source, I would recommend obtaining it from this location instead. Here, the data is organized into training, validation, and testing sets. The data structre is described in the small README.txt that is available, there:
I'm sorry, but I'm not entirely sure about the specific point you're referring to. Would you kindly provide me with a hint or more details to help me better understand? |
My intention was solely to add the |
added --verbose parameter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @ThomasFroech sorry about the confusion about the update_probs()
- that change came from an unrelated commit.
- Added documentation based on info provided.
- Fixed style.
added dataset classes for the TUM-Facade dataset (https://github.com/OloOcki/tum-facade ) and updated the init file
This change is