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

v0 of find_minimal_coverage #510

Merged
merged 10 commits into from
Dec 5, 2024
Merged

v0 of find_minimal_coverage #510

merged 10 commits into from
Dec 5, 2024

Conversation

fcollman
Copy link
Collaborator

@fcollman fcollman commented Dec 3, 2024

not completed yet complete, but scaffold of endpoint to find minimal set of nodes that covers all passed in nodes

@fcollman
Copy link
Collaborator Author

fcollman commented Dec 4, 2024

I think this is done now.

@fcollman fcollman force-pushed the cover_minimal_nodes branch from b66cdb1 to e1288ab Compare December 4, 2024 17:24
@@ -11,7 +11,7 @@ flask
flask_cors
python-json-logger
redis
rq
rq<2
Copy link
Collaborator Author

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.

Copy link
Contributor

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.

Copy link
Contributor

@akhileshh akhileshh left a 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

@fcollman fcollman merged commit 696925e into main Dec 5, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants