-
Notifications
You must be signed in to change notification settings - Fork 10
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
v0 of find_minimal_coverage #510
Conversation
I think this is done now. |
b66cdb1
to
e1288ab
Compare
@@ -11,7 +11,7 @@ flask | |||
flask_cors | |||
python-json-logger | |||
redis | |||
rq | |||
rq<2 |
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 noticed this was a problem when trying to get a local version of PCG running and so i added it here pre-emptively to save us time in the future when we try to recompile reqs.
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 think this is fine for now, version 2 was released only recently in Oct.
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.
looks good except for the print statement
not completed yet complete, but scaffold of endpoint to find minimal set of nodes that covers all passed in nodes