Added the poetry lockfile (this doesn't need reviewing)
Folder structure
Renamed to codonpython (it didn't seem to like the camel case)
Updated folder structure to match typical package format (/src/packagename, and /tests/)
Updates to deprecated code
Updating the dependencies meant some of the older deprecated code didn't work. Some fixes:
One-line change to a pandas df method (when passing aggregate function to df.groupyBy().agg(), you used to pass a built in function, e.g. sum, but that has since been deprecated. Now you need to pass the name of the function as a string e.g. "sum")
Changed use of the deprecated df.append() to the newer pd.concat() in one place
Changed pandas.util.testing.assert_frame_equal to the newer pandas.testing.assert_frame_equal
Tweak to tests
Added a path to the file import function tests. This forces them to scan the tests/ folder instead of the whole project directory, since these might indeed have csvs in them and cause the tests to fail. We will later need to add test csv and xlsx files in the tests/ folder, but that can be another ticker, I have made issues for these
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
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.
Changes
Packaging
Folder structure
Updates to deprecated code
Updating the dependencies meant some of the older deprecated code didn't work. Some fixes:
Tweak to tests
Set up
Ignore the readme! :)
If codespaces, do:
This will install dependecies and the current package.
Then you can do:
It is published on Test PyPi so you could also do:
and then for example: