Update dependencies and pyproject.toml
#432
Open
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.
In the process of running a data update I thought it would be helpful to update some dependencies and simplify where we name dependencies. Initially it was spread between
requirements.txt
andsetup.py
. Butsetup.py
was recently removed. I thought it would be simpler if everything was inpyproject.toml
. This PR makes the following changes:requirements.txt
in favor of putting all dependencies underpyproject.toml
. It seems like the main advantage ofrequirements.txt
was having one place where Docker could install all development dependencies, but I instead had Docker just download thedbcp
package + all the dependencies inpyproject.toml
. I'm not that familiar with Docker so I'm not sure if this is problematic or not the standard convention.Remaining To Do:
python 3.12
andpandas 2.0