We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Description of the desired feature
Continuing on from #1276, there are some parameters in upstream GMT at https://docs.generic-mapping-tools.org/6.2/grdfill.html which are still missing in PyGMT at https://www.pygmt.org/v0.4.0/api/generated/pygmt.grdfill.html. Your task is to fill them in!
pygmt/pygmt/src/grdfill.py
Lines 21 to 22 in d90b3fc
Lines 54 to 55 in d90b3fc
You'll need to change two parts of the grdfill.py module like so:
R="region", V="verbose", f="coltypes",
{R} {V} {f}
grdfill
Tip: See the Pull Request at #1343 for an example of how a common alias can be added.
Suggest a good long alias name for the grdfill -N parameter, and add it to the Pull Request!
grdfill -N
🥳 Good luck, and let us know if you need any help :)
Are you willing to help implement and maintain this feature? Happy to review someone's Pull Request on it.
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
Description of the desired feature
Continuing on from #1276, there are some parameters in upstream GMT at https://docs.generic-mapping-tools.org/6.2/grdfill.html which are still missing in PyGMT at https://www.pygmt.org/v0.4.0/api/generated/pygmt.grdfill.html. Your task is to fill them in!
Current state 📝
pygmt/pygmt/src/grdfill.py
Lines 21 to 22 in d90b3fc
pygmt/pygmt/src/grdfill.py
Lines 54 to 55 in d90b3fc
Desired state ✨
You'll need to change two parts of the grdfill.py module like so:
How to make the change 🧐
grdfill
module's docstring, go to https://github.com/GenericMappingTools/pygmt/edit/master/pygmt/src/grdfill.py and add in the new docstring!Tip: See the Pull Request at #1343 for an example of how a common alias can be added.
Stretch goals 🚀
Suggest a good long alias name for the
grdfill -N
parameter, and add it to the Pull Request!🥳 Good luck, and let us know if you need any help :)
Are you willing to help implement and maintain this feature? Happy to review someone's Pull Request on it.
The text was updated successfully, but these errors were encountered: