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

-R is required in grdproject? #8273

Closed
seisman opened this issue Jan 11, 2024 · 7 comments
Closed

-R is required in grdproject? #8273

seisman opened this issue Jan 11, 2024 · 7 comments
Labels
bug Something isn't working
Milestone

Comments

@seisman
Copy link
Member

seisman commented Jan 11, 2024

gmt grdproject @earth_relief_01d_g -JH10c -Gtmp.nc

The above command used to work in GMT 6.4, but reports an error in GMT 6.5:

grdproject [ERROR]: Option -J: If map width is given you must also specify a region with -R

but the documentation says:

Specify the region of interest. You may ask to project only a subset of the grid by specifying a smaller input w/e/s/n region [Default is the region given by the grid file].

Then, why -R is required here?

@PaulWessel
Copy link
Member

Vague memory says we did this to fix another issue. Within last 1 year? Maybe you can find the commit and we can see what we were trying to fix.

@seisman
Copy link
Member Author

seisman commented Jan 11, 2024

it was added in #7974

@joa-quim
Copy link
Member

But what is the meaning of passing a JH10c? It works with -Jh

@seisman
Copy link
Member Author

seisman commented Jan 12, 2024

But what is the meaning of passing a JH10c? It works with -Jh

I've never used this module, so I have no idea what -JH10c exactly does, but it's allowed and also used in the test (https://github.com/GenericMappingTools/gmt/blob/master/test/grdproject/units.sh).

@PaulWessel
Copy link
Member

Would like to find the commit the added this - are you able to help with that @seisman ? My git skills are deteriorating... I am 95% sure we did this to fix another problem but cannot recall what that was.

@seisman
Copy link
Member Author

seisman commented Jan 12, 2024

It was added in this PR #7974.

@seisman seisman added this to the 6.6.0 milestone Jan 12, 2024
@seisman
Copy link
Member Author

seisman commented Jan 12, 2024

Closed by #8286.

@seisman seisman closed this as completed Jan 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants