Default to color bars with gaps for categorical CPTs#7831
Conversation
See #7830 for backfround. This PR implements the change. Old (and in our opinion wrong) behavior can be obtained by just giving -L with no or zero gap.
|
Sure, can see why. But what is a good solution here? This is a page of all categorical CPTs but we treat them as any other CPT. Perhaps it should look different since it is categorical and the script has to adjust. In M_1a we have the paired.cpt who is the same kind, but it is also placed in M_1c so we should remove it so it does not show twice. For all of those the "top continuous, bottom discrete" makes no sense anyway so best to just show those in a separate (new) format. Agree? Thoughts on this, @joa-quim ? |
|
@Esteban82, could you please send me the Rust to Silver CPT from Mercury(?). I cannot find it but I know I kept it.... |
|
I have never played with the categorical before and so I'm in the learning process. My doubt is even more basic. What are those SCM categorical cpts good for? They look absolutely useless to me. Who wants 100 classes? And if we interpolate them to small number (8 in the bottom palettes) the result is garbage with several classes with indistinguishable colors. Edit: sure they look garbage. Categorical cmaps are not interpolable |
|
All true. I am reducing to 25 cases and 12 for our paired.cpt. We are just passing SCM to users who hate rainbows and turbo. |
|
Right for the SCM's, but the SCM's-S (the categoricals) are garbage to me (until I'm instructed on its validity) and we should simply drop them, as well as the |
|
No reason to do so. It does let people see them. |
|
They can be see in |
|
I sense your anger so WTF, we will trash 1c. I am angry too, spent 90 minutes on fixing that plot! |
|
You should know that Sunday lunches here last for long. You should had asked my thoughts only after I had time to do a little siesta 😄 |
Hi Paul. Here is the CPT (I add the extension .txt so I can upload it here). RustToSilver_180207.cpt.txt I used this script to create it from the original CLR file. If you want to use it for pluto, them you only will have to adjust the values (I think). |

See #7830 for background. This PR implements the change. Old (and in our opinion wrong) behavior can be retained by just giving -L with no or zero gap.
Part 1 (implementation) is done
Part 2 (hence WIP) is to update some scripts that currently plot no gaps. I will let a few of those add **-**L so the PS does not change but at least one should give the new behaviour and I will dvc that plot only.
Closes #7830.