Skip to content
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

Make all plotting dependencies optional #117

Open
ocefpaf opened this issue Jul 4, 2024 · 3 comments
Open

Make all plotting dependencies optional #117

ocefpaf opened this issue Jul 4, 2024 · 3 comments

Comments

@ocefpaf
Copy link
Member

ocefpaf commented Jul 4, 2024

We should make all plotting dependencies, like pandas_flavor, matplotlib, and cartopy (any other?), optional as a plotting extras.

@FloraSauerbronn this would require you to edit the requirements.txt file, to remove any plotting dependencies that may be listed there, and add a new entry in the pyproject.toml like:

[project.optional-dependencies]
plotting = [list_the_dependencies_here ]

Can you tackle that?

@neerja-1984
Copy link

@ocefpaf went through the above issue .. is it sorted out ?

I wish to contribute to the repo and was anticipating any ideas / enhancements iooc is looking in the project

@ocefpaf
Copy link
Member Author

ocefpaf commented Jan 21, 2025

Glad you are interested. Can you check the dependency descriptions and figure out if these is done or not? It would be a way to show your proficiency with pyproject.toml and Python dependency management.

@neerja-1984
Copy link

@ocefpaf this is my first time working with .toml files
have made this pr as per my understanding #134

do let me know if it meets the requirement

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants