Palette cli

[justinlaughlin]

• Adds the option to specify a palette using CLI arg -pname or --palette-name
• Adds option to specify an external palette file and/or palette name to use in glvis "scripts" and streams
• Internally, use PaletteState.SetFallbackIndex() over PaletteState.SetIndex() to specify a fallback index if the default is not specified (implementation would have been more invasive via PaletteState so it is done through the global BasePalettes)

[najlkin]

How about script and stream commands. Could you add them too? 😉 If you branch this from #335 , it would be very easy. It makes sense especially with palette files, because it is not that easy to sort out what will be the index when loaded to GLVis.

[tzanio]

@justinlaughlin, can you resolve the conflicts?

[justinlaughlin]

Updated to branch off of #335 ; I added script/stream commands. ping @najlkin @tzanio

To test the script I ran this script with mesh/solution generated from a mfem example

solution refined.mesh sol.gf
palette_file share/palettes-crameri.txt
palette_name batlow

To test the stream I modified capacitor.saved and added this at the end

palette_file share/palettes-crameri.txt
palette_name batlow

[najlkin]

A rather tangential, but also when you change the palette, the number that is shown is off by one compared to the list 🫣 .

[najlkin]

Everything seems to work! 🤩

[najlkin]

Looks good! 🎉 Pls mention it in changelog.

[najlkin]

Ahh, I do not like that "palettes", that "palette_file" was better in my opinion 😛 I do not think that there must be consistency with command line parameters. This is rather confusing, because "palette" command takes the index and "palettes" a file 🤪 .

[tzanio]

Thanks @justinlaughlin and @najlkin !