-
Notifications
You must be signed in to change notification settings - Fork 342
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
CPT to plot magnetic anomaly maps. #1488
Conversation
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.
Lots more work than that to add an extra CPT. There is gmt_cpt_masters.h and GMT_App_M_1?.sh, etc. I dont think we should ship undocumented CPTs hidden in share. You can do that in the cache.
The |
And speaking of new CPT's. Shouldn't we add the new rainbow from Google that I mentioned you? |
There is room for adding a pair of CPTs so if you get the google one too. However, that one has no hinge and should be normalized to 0-1 in z (see others). I think the mag has a hinge (zero mag) but no zero entry in the CPT. It would probably be best to split that -50 to 50 up in to -50 to 0 and 0 to 50 for that reason, and add the HINGE flag. |
OK, what do we call the google cut? google? spectrum? Once you add it I can go to work on App M. |
Well, they called it turbo. It's a stupid name but that's how it's going to be known. |
OK, turbo is fine since it is not already used. |
Need entry in gmt_cpt_masters.h for turbo. |
Done. I think this turbo is a candidate to a new default CPT (if that is still on the table) |
So some difficulty with App. We have 71 CPTs in gmt_cpt_masters.h, but one we skip (srtm which is same as dem4 on land only). So 70 minus the 24 from Fabio = 46. To split into two pages with two columns that number needs to be divisible by 4. So we actually need 48. Are we able to come up with 2 more CPTs. |
How about cequal [http:https://soliton.vm.bytemark.co.uk/pub/cpt-city/imagej/tn/cequal.png.index.html] and columbia [http:https://soliton.vm.bytemark.co.uk/pub/cpt-city/wkp/shadowxfox/index.html]? |
And what about dropping 2? We already have more than enough generic ones. |
Would give backwards compatibility headaches. Better to add two. OK, could remove two from the table but leave in share - that is backwards compatible since the gmt_cpt_masters.h is only used for listing (and by App M). What to remove? |
Maybe seafloor and wysiwyg? |
wysiwyg is a good candidate. And the acton? We only break some compat if remove old ones. New ones were technically never really released. |
acton is part of Fabio so does not help with the 24 problem. |
I think seafloor is pretty useless and there is ibcao, bathy, abyss already. |
Yes, we have several seaflors. Panoply seems pretty useless too. |
I'll remove seafloor and wysiwyg. Remko loves panoply... |
Why do we have to remove two CPTs? Can we list all 70 cpts in one long paper? Or can we have 24, 22, and 24 cpts in three pages? |
OK, done and updated M 1b PS. Now for discussion of turbo vs rainbow. I would think we agree that turbo is a much improved rainbow without being washed out as in Fabio, no? Maybe @seisman might agree as well? |
Words of wisdom from @seisman. We don't have to make 3 separate M plots. We could keep the Fabio stuff separate and then do everything else as a longer page. We are no longer dealing with page plotting. I will see what I can do - the App M script is messy... |
I once tried to rewrite the three App M scripts with subplot. It's more readable but much slower. |
Got them all to fit in APp M.
I got things to fit as @seisman suggested with 24, 22, 24 on three pages. |
Changing to turbo will probably affect some of our PS origs. |
I have no objection. |
Makes it easier to change the default palette in the future.
I'm very favorable. It's not only the perceptual thing but also that it drops the violet, which always felt wrong on topography maps. |
Running all tests but having trouble: psxy [ERROR]: Unable to down file http:https://oceania.generic-mapping-tools.org/gmt_md5_server.txt Not sure why this is happening but I have seen it before. So lots of failures obviously if we cannot get the cache files. |
We are finding out that possibly due to a UH firewall change, nodes INSIDE the UH network cannot access oceania but nodes on the outside can. The SOEST IT staff says it has nothing to do with the SOEST firewall, but there seems to be some infinite loop being set up when we try within UH. |
I'm back now. Should I push the updated PS files to this branch? |
Yes please, I am unable to access the cache since I am inside the UH network... |
Updated 10 PS files. |
So if tests pass for you then I guess we can merge this PR? |
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.
Good to merge.
Implemented. |
It's the one used in IPGP for decades