Modernize dependency management#170
Draft
chimaerase wants to merge 1 commit intoEpistasisLab:mainfrom
Draft
Conversation
Author
|
@perib or others, I'm uncertain about notifications you'll receive related to draft PR's, but here it is. |
a67f7a0 to
b520b00
Compare
Author
|
Force pushing again to pull in lessons learned from another project:
I probably can't contribute much more to this, but again hope it's a useful starting point. |
b520b00 to
0266783
Compare
jamesmyatt
reviewed
Apr 17, 2025
Author
|
Force pushing to fix typo, and change "gpu" term to "cuda" to match other repos. I'll also update the PR description to matche. |
0266783 to
c3514ed
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Provide baseline updates to use
poetryfor more automated dependency management, as well as differentiating between CPU- and GPU-targeted installs. Likely some details are missing here, but I'm hoping this is a useful baseline to work from that resolves several issues:New local dev install process would be, e.g.:
poetry install --with cpu
or poetry install --with cuda
or poetry install --with test
Downstream pypi downloads of published versions would support extras, e.g.
pip install tpot2[cpu]or similar.Where should the reviewer start?
pyproject.tomlHow should this PR be tested?
With TPOT's existing CI/CD or internal test process, which I'm unfamiliar with nuances of, but I hope this is a big boost in a helpful direction. Unfortunately, I'm at the limit of time I can justify spending on this, but I hope this is a helpful starting point!
What are the relevant issues?
#146
Screenshots (if appropriate)
Questions:
Yes
No