Extend do_nd with option to perform "Parallel sweeps"

Open jenshnielsen opened this issue 3 years ago • 1 comments

In the process refactor Sweeps and Measurements to be a bit more modular

TODO

[ ] More robust parsing of Parameters (see todos in code)
[ ] Document
[ ] More complete tests

Jun 29 '22 08:06 jenshnielsen

Codecov Report

Merging #4325 (523a063) into master (c4ccf28) will increase coverage by 0.04%. The diff coverage is 98.62%.

@@            Coverage Diff             @@
##           master    #4325      +/-   ##
==========================================
+ Coverage   68.28%   68.32%   +0.04%     
==========================================
  Files         339      339              
  Lines       31794    31990     +196     
==========================================
+ Hits        21710    21857     +147     
- Misses      10084    10133      +49

Aug 03 '22 13:08 codecov[bot]

Should add a test that we get a resonable error message if there are no measurement parameters

def test_dond_without_measurement_params():
    a = ManualParameter("a", initial_value=0)

    with pytest.raises(
        ValueError
    ):
        dond(ArraySweep(a, [1, 2, 3]))

See #4366

Sep 23 '22 14:09 jenshnielsen

bors merge

Sep 30 '22 12:09 jenshnielsen

Build succeeded:

Sep 30 '22 12:09 bors[bot]