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

catch unbound localerror #20

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

Conversation

richardrl
Copy link
Contributor

No description provided.

Copy link
Collaborator

@vitchyr vitchyr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to have a bunch of other changes other than the unbound local error. Can you separate these things into separate PRs? The default of not using a GPU is reasonable, but I'm not sure (e.g.) why we should move the scripts into their own module or comment out the code that checks if launching EC2 is okay.

@richardrl
Copy link
Contributor Author

Updated to only include the gpu and unboundlocalerror

@richardrl
Copy link
Contributor Author

Oh, there's also the movement of the scripts folder. I made an issue about it. If thats intended and something else is going on, I can change that

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.

2 participants