-
Notifications
You must be signed in to change notification settings - Fork 167
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
Minimal Pyjanitor instalation #826
Comments
possible dup of #793 |
Hello @GuiMarthe! Thanks for chiming in. Yes, there are a lot of dependencies for pyjanitor. I think the dependency sprawl has been something I haven't managed well in the past, still doing a bit of learning here. Looks like it might be good for us to split out at least the Would you like to help contribute a PR, if you've got the bandwidth? Meanwhile, I'll tag this and the other #793 as being one of the higher priority issues. |
hey @ericmjl, thank you for the prompt response! I think I can help you with this issue, even though I'm a beginner, I am aware of how friendly pyjanitor is 😄 . Now, I'd still need to dig into the code, but from the
Perhaps the ML category could be merged with the base category. Does that make sense? |
Also, should we add messages and/or warnings whenever the user tries to use a method that depends on a non-installed dependency? |
@GuiMarthe yesss! That sounds like a great idea.
That makes sense too. In terms of the different ways we can group things, would you be kind enough to do the following?
Doing so would mirror the structure in the I'm looking forward to reviewing the PR! I will be getting my vaccine shot this week, so I might be KO'd for a day or two (depending on whether my immune system kicks off in a big or small way), but I should be able to come back to it later. |
Would |
Actually, that sounds like a good idea, @hectormz! @GuiMarthe, can I check in, do you have bandwidth to handle this one? I ran into a bit of a busy patch myself, and have dropped the ball here, and probably will be like this until the end of the week. |
Hey folks, I recently looked at the package dependency for pyjanitor and it seems too large for a production environment.
I saw in the
.requirements
directory that there are a few sets of dependencies for different use cases, but I don't see anywhere how to actually limit the scope at installation time.Is the following possible?
pip install pyjanitor[base]
Just in case, this is the packages dependency list generated by
pipdeptree
.If I'm doing anything wrong, please let me know!
The text was updated successfully, but these errors were encountered: