You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For 'mode', 'auto' (mode 4) and for non-powder mode, sw_plotspec tries to determine what type of spectra is being plotted and then calls itself again with 'mode', 'disp' or 'mode', 'color', however this second call ignores certain user arguments without warning.
Specifically:
For spectra without swConv (e.g. not run through sw_egrid), sw_plotspeccalls itself again with 'mode', 'disp' but overrides every argument except for qlabel.
For spectra with swConv, it calls itself again with 'mode', 'color' but overrides dashed, and colorbar.
Behaviour 2 is ok, but behaviour 1 is quite annoying and unexpected for users.
sw_plotspec should be refactored to either:
Avoid this recursive call (e.g. just set mode to 'disp')
Or emit a warning that user provided arguments will be ignored.
The text was updated successfully, but these errors were encountered:
* Fix bug where imag always plots in auto mode #172
* Allow row vector for hkl for single point #179
* Fix inverted user colormap #131
* Refactor sw_plotspec to avoid recursive calls #132
* Add fitspec fixes suggested by K Richardson #98
* Add spinw obj to spec for twin cases in fitspec #158
* Fix spec spinw object issues in fitmode
spinwave() now creates a .obj field regardless
When fitmode=true, this a barebones struct
When fitmode=false, this is a full spinw object
Fix issue with sw_plotspec when using fastmode/fitmode
Add basic test for fitspec() to check twins handling
* Fix bug in sw_plotspec 'fastmode' handling
* Fix issues from review. Add change log.
For
'mode', 'auto'
(mode4
) and for non-powder mode,sw_plotspec
tries to determine what type of spectra is being plotted and then calls itself again with'mode', 'disp'
or'mode', 'color'
, however this second call ignores certain user arguments without warning.Specifically:
swConv
(e.g. not run throughsw_egrid
),sw_plotspec
calls itself again with'mode', 'disp'
but overrides every argument except forqlabel
.swConv
, it calls itself again with'mode', 'color'
but overridesdashed
, andcolorbar
.Behaviour 2 is ok, but behaviour 1 is quite annoying and unexpected for users.
sw_plotspec
should be refactored to either:'disp'
)The text was updated successfully, but these errors were encountered: