-
Notifications
You must be signed in to change notification settings - Fork 224
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
Add sequence_space converter in kwargs_to_string #325
Add sequence_space converter in kwargs_to_string #325
Conversation
Introduce new 'sequence_space' converter for transforming a Python list or tuple into a space separated string. Useful in cases for e.g. when multiple file inputs can be passed into some module. Also added a doctest example into the kwargs_to_string function.
While working on Might open another PR on this so that users can do |
It should already works, as we already can have multiple -B options. import pygmt
fig = pygmt.Figure()
fig.basemap(region="0/10/0/10", projection="X10c", frame=["xaf", "yaf", "+tTitle"])
fig.show() |
Wow, I never knew that worked! Must have missed that in one of the tutorials 😱 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Feel free to merge it @weiji14
Using the sequence_space processor in #325 to allow for passing in a list of textfiles to be plotted! Problem with that `text` would consider a numpy matrix as a list to be parsed, so the code was updated to fix that. Unit tests and docstrings updated. Strangely enough, the 'projection'/'-J' argument doesn't seem to be required anymore? Also regenerated baseline plots to use their rightful name...
Description of proposed changes
Introduce new 'sequence_space' converter for transforming a Python list or tuple into a space separated string. Useful in cases for e.g. when multiple file inputs can be passed into some module.
For example:
Fixes #322
Reminders
make format
andmake check
to make sure the code follows the style guide.doc/api/index.rst
.