Open mathieuboudreau opened 1 month ago
It all looks great to me, thanks!
Before merging, wanted to give you heads up that the deploy workflow on main sets
mb/dev
as trigger ref
Yep thanks! I was about to change it to main again last night but fell asleep haha - will do asap
Thanks all! I was about to merge when I noticed the colors are going the wrong way (blue should be negative and red positive) - I think plotly has two different definitions for this diverging colormap - I'll look up the other one and fix it prior to merge!
the colors are going the wrong way
Good catch!
Following up on #12, I'm suggesting to switch the B0 maps from grayscale to a diverging colormap (I have a preference of blue-white-red since it's intuitive), with white set to 0, blue negatives, red positives. I'm open to a discussion, how are they being displayed in Shimming-Toolbox?
Here's what they look like with my changes,
Figure 5.1
Figure 5.3
Figure 5.9