search

OxfordIonTrapGroup / ndscan

N-dimensional scans for ARTIQ
GNU Lesser General Public License v3.0
18 stars 15 forks source link

plots: adjustable num_samples_per_point #407

Open pmldrmota opened 4 months ago

pmldrmota commented 4 months ago

Added QSpinBox to context menu so that user can get more accurate error bars when averaging.

image

dnadlinger commented 4 months ago

I would like to improve the user experience a bit here, in the sense of making it clear to the average physicist what is actually going on here. If I hadn't thought about this particular problem already, I wouldn't have a clue what "samples per point" actually means. Perhaps we can make the label "Assume[d] samples per point" instead, to make clear that there are no samples involved in the averaging process itself? We could also add a tooltip with a brief explanation, and should probably move the discussion in the docstring to some user-visible documentation (maybe just a new "the ndscan applet" page?).

Separately, I wonder whether it would be worth introducing a way to include enough information about the result distributions to do averaging and similar operations correctly directly with each result channel (either as channel metadata, or some kind of "rich FloatChannel").