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

changing print to logging and minor logging fixes #14

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

Conversation

gayalkuruppu
Copy link

I have changed all the print statements to logging, for it to be able to run using nohup command. I have also changed some existing logging.info() lines which have given tuples where logging.info() expects strings in python 3.

gayalkuruppu and others added 10 commits October 19, 2020 22:34
When using this file (running the process_polyvore.sh) in cloud it is better to use nohup command. For that I added logging instead of print
When using this file (running the process_polyvore.sh) in cloud it is better to use nohup command. For that I added logging instead of print
When training in the cloud ssh might broke, so it is always better to use nohup command. So I added logging instead of print, to log those print statements.
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

Successfully merging this pull request may close these issues.

1 participant