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

Replace color for fill in PyGMT plots #449

Closed
santisoler opened this issue Jan 9, 2024 · 0 comments · Fixed by #495
Closed

Replace color for fill in PyGMT plots #449

santisoler opened this issue Jan 9, 2024 · 0 comments · Fixed by #495
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers (doesn’t require deep knowledge of the project)

Comments

@santisoler
Copy link
Member

Description of the problem/suggestion:

While building the docs, PyGMT is raisin a FutureWarning about the deprecation of the color argument in favor of the fill argument:

FutureWarning: The 'color' parameter has been deprecated since v0.
8.0 and will be removed in v0.12.0. Please use 'fill' instead.  

We should update the User Guide that generate plots using the color argument and use fill instead.

Tip

Good first issue
This issue is a nice way to start contributing to Fatiando. If you would like to start contributing to the project and learn more about the GitHub workflow, this issue is a nice way to start. Feel free to comment below if you are interested in solving it.

Don't hesitate to do it if you are interested. You won't be alone, we'll provide as much as help as needed to guide you through it!

@santisoler santisoler added good first issue Good for newcomers (doesn’t require deep knowledge of the project) documentation Improvements or additions to documentation labels Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers (doesn’t require deep knowledge of the project)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant