-
Notifications
You must be signed in to change notification settings - Fork 28
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
Stack height classifier #1
base: grasp_pytorch0.4+
Are you sure you want to change the base?
Conversation
…r_visual_stacking into grasp_pytorch0.4+
…+' into grasp_pytorch0.4+
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.
Aside from the logger this mostly seems fine for now. Can you explain the second logger file?
logger_for_classifier.py
Outdated
import csv | ||
# import h5py | ||
|
||
class Logger(): |
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.
Why is a whole copied (and modified?) Logger class needed for the classifier? This does not seem like the right way to implement this.
Is there a way the existing logger can be used, perhaps with minor modifications?
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.
i created another logger.py thinking it won't be part of the existing branch.
But yes, it is better to use the existing logger with some changes.
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.
I deleted the second logger file and used the existing logger.py file
Hey any updates on the classifier component? |
add primesense code source links
Stack height classifier data generating script