Context:
Now that the visualization improvements have been added to MrMustard, there's some feedback that needs addressing. This PR does that.
Description of the Change:
Change the default colour scheme in the 2D/3D Wigner plots from viridis to RdBu to distinguish from the density matrix plot as suggested in #418. I definitely agree with the choice - negative is red, zero is white, and positive is blue. Felt semi-intuitive. If you'd prefer another one from plotly's list, lmk!
Force the mid-point of the colour axis in those Wigner plots to always be 0 instead of auto-scaling, and show the scale
Change the widget in the Fock repr to be kinda-square
Change the 2D Fock magnitude plot to be a bar graph
Benefits:
Things look more how we want them to
Wigner color scale is consistent between plots to avoid confusion, and scale is shown now
Fock magnitude plot no longer falsely suggest that it represents continuous information
Possible Drawbacks:
Maybe you don't like RdBu?
Maybe you'd prefer the colour scale to be hidden like before?
Context: Now that the visualization improvements have been added to MrMustard, there's some feedback that needs addressing. This PR does that.
Description of the Change:
viridis
toRdBu
to distinguish from the density matrix plot as suggested in #418. I definitely agree with the choice - negative is red, zero is white, and positive is blue. Felt semi-intuitive. If you'd prefer another one from plotly's list, lmk!Benefits:
Possible Drawbacks:
Related GitHub Issues: Fixes #417, Fixes #418