-
Notifications
You must be signed in to change notification settings - Fork 1
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
Merge data-file-summarizer into overviewpy #17
base: feature-summarizer
Are you sure you want to change the base?
Conversation
…n__ into overviewpy directory.
This PR is a work in progress. Additional work is needed to:
|
…ackage installed with pip.
…ng jinja2 as a dependency.
Ok, I've realized there already is a code method to access the summarizer without dropping back to the command line. So I'm caling that one done. For the last one, I kind of mislove the use of I propose merging the PR as it is and we can adjust code style and stuff after the fact. |
Asking around a bit on masstodon, seems like most of the community sees no problem with aliasing pandas as pd. So, since the community is down with it and the rest of the package already uses that nomenclature, I bit the bullet and updated the summarizer to use that convention also. |
Thanks again for the PR! I changed the target branch - this way we can merge all upcoming PRs to an up-to-date base feature branch that mirrors the main branch, and make sure all changes are done as expected before moving them back to Just wondering because there seem to be issues with |
I'm developing on OSX in a conda environment with python 3.10. What kind of problems are you seeing with poetry? |
As discussed in Issue #14, moving the functionality of data-file-summarizer into overviewpy.