-
Notifications
You must be signed in to change notification settings - Fork 339
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
inset -R -J -X -Y history #6288
Comments
... nevermind ... sorry |
Ok ... I've retried ... it does conserve -X -Y but neither -R or -J
|
It should retain -R -J once set inside the inset box. Could you help by supplying a full test script? |
If I interpret correctly the black line placement, it seems that -JG0/0/3c is not respected (it takes 3.5c) |
I think for gmt inset you have to stick with the rules for the inset:
A successful script based on what you have above does not use -X -Y at all and looks like this:
Yet there are clearly documentation issues here, so I am not quite sure what is the best way forward here. Here are a bunch of concerns or issues.
My proposal is this:
|
While the -X and -Y options serve no purpose in the gmt inset call, and are not listed as allowable options neither in synopsis nor documentation, we did in fact not prohibit them, leading to troubles (#6288). This PR slams that door shot. No example or tests used -X -Y since that was never the plan.
So I would have to use something else than |
While the -X and -Y options serve no purpose in the gmt inset call, and are not listed as allowable options neither in synopsis nor documentation, we did in fact not prohibit them, leading to troubles (#6288). This PR slams that door shot. No example or tests used -X -Y since that was never the plan.
I am not completely done with this issue yet, but the intent is that you should be able to use -X -Y on commands inside your inset begin/end clause, but not with inset itself. There may be issues related to this as well but that is for the next steps. The key issue here is that gmt inset itself was never meant to take -X -Y. |
There's an issue with ex44.sh after the recent inset changes. It's odd because the script does not use -X or -Y, but here are the error messages:
|
Thanks, I will look at it later today or the weekend (busy day for other matters) |
Thanks @PaulWessel, good luck ! |
Not sure it is a bug but it seems that
inset
doesn't retain history within its block. Is it on purpose ?Guillaume
The text was updated successfully, but these errors were encountered: