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.

Sign up for GitHub

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

Jump to bottom

Move grape installation for QC report to long lived branch #35

Open
amc-corey-cox opened this issue Oct 3, 2022 · 1 comment
Open

Move grape installation for QC report to long lived branch #35

amc-corey-cox opened this issue Oct 3, 2022 · 1 comment
Assignees
@amc-corey-cox

Comments

@amc-corey-cox
Copy link
Collaborator

The installation of grape is too difficult on some systems (Windows) for the gains we're seeing right now so I'm moving it to a long-lived branch for re-integration next time.
@amc-corey-cox
Copy link
Collaborator Author

Moved grape-related code to a single file (except commented portions) and removed from poetry environment. Chages are merged so we now no longer have a dependency on grape.

Still need to create long-lived branch with removed changes to bring back later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@amc-corey-cox amc-corey-cox

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@amc-corey-cox