Ticket #8597 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

[Muon] MuonLoad workflow algorithm

Reported by: Arturs Bekasovs Owned by: Arturs Bekasovs
Priority: major Milestone: Release 3.1
Component: Muon Keywords:
Cc: Blocked By:
Blocking: #8494 Tester: Roman Tolchenov

Description

This should encompass all the operations done to the workspace by MuonAnalysis from the moment the workspace is loaded to the moment it gets plotted.

Change History

comment:1 Changed 7 years ago by Arturs Bekasovs

  • Status changed from new to inprogress

Refs #8597. Initial class files.

Changeset: 829b0993680f7c2fab12af8e56261738223d2baa

Last edited 7 years ago by Arturs Bekasovs (previous) (diff)

comment:2 Changed 7 years ago by Arturs Bekasovs

Refs #8597. Initial documentation and properties.

Changeset: 153238e10deded09cdecd101fbfa59079359e71b

comment:3 Changed 7 years ago by Arturs Bekasovs

Refs #8597. Test for basic functionality.

Changeset: f8b41819012053d2ea02171f340dbbaf01462e41

comment:4 Changed 7 years ago by Arturs Bekasovs

Refs #8597. Initial logic.

No grouping, DTC or any other corrections applied yet.

Changeset: 3c9d43fedbc1e440757489609d470ce3ed5c0147

comment:5 Changed 7 years ago by Arturs Bekasovs

Refs #8597. Test for multi-period data.

Changeset: 4f290916d0f39c0df54dcd0823733dc6cd01bf11

Last edited 7 years ago by Arturs Bekasovs (previous) (diff)

comment:6 Changed 7 years ago by Arturs Bekasovs

Refs #8597. Bin value correction.

Changeset: 7810548d3ddfd8bcc7cfd8a1ffb09202193e59fb

comment:7 Changed 7 years ago by Arturs Bekasovs

Refs #8597. Dead time correction functionality.

Changeset: b70a3b288faee9c596c0618620352f559549270f

comment:8 Changed 7 years ago by Arturs Bekasovs

Refs #8597. Simple test for error reporting.

Changeset: 7d4eac1c5dfcb81b00929f538ef927e7157b2896

comment:9 Changed 7 years ago by Arturs Bekasovs

  • Status changed from inprogress to verify
  • Resolution set to fixed

Tester:

Please don't get confused by the branch name - I've mixed up the ticket numbers.

The main thing to check - MuonLoad algorithm should produce the same plots as the ones produced by MuonAnalysis interface with the same parameters. Some of the functionality is currently covered by unit test, more will be covered by SystemTest, so this is just a sanity check.

  1. Open MuonAnalysis interface. Load file: AutoTestData/MUSR00015189.nxs
  2. Set the following parameters:
    1. Home tab. Time Zero: 0.50, First Good Data: 0.1. Dead Time Correction: From data file, Period algebra: "2-1"
    2. Settings tab. Time axis: Start at first good data, Finish box: 16, Data binning: Fixed, 2 steps
  3. Plot the 'long' pair.
  4. Now, launch LoadMuonNexus, set DetectorGroupingTable to Grouping and load MUSR00015189.nxs file. Workspace Grouping should be created.
  5. Open MuonLoad algorithm dialog.
  6. Set properties:
    • Filename: MUSR00015189.nxs
    • First Period: 1 (= 2 on the interface)
    • Second Period: 0 (= 1 on the interface)
    • PeriodOperation: -
    • ApplyDeadTimeCorrection: true
    • DetectorGroupingTable: Grouping_1
    • TimeZero: 0.5
    • RebinParams: 0.032 (not 2 because MuonAnalysis interface actually multiplies specified number by the bin width, in this case - 0.016)
    • Xmin: 0.1
    • Xmax: 16.0
    • OutputType: PairAsymmetry
    • PairFirstIndex: 0 (corresponding to group fwd)
    • PairSecondIndex: 1 (corresponding to group bwd)
    • Alpha: 1.0
    • Output workspace: Result
  7. Run the algorithm.
  8. Check that plot of workspace "MUSR00015189; Pair; long; Asym; #1" produced by MuonAnalysis corresponds to the plot of "Result" workspace.
  9. Try changing the parameters of the MuonLoad and check that they have expected effect. You can check the results using MuonAnalysis interface.
Last edited 7 years ago by Arturs Bekasovs (previous) (diff)

comment:10 Changed 7 years ago by Roman Tolchenov

  • Status changed from verify to verifying
  • Tester set to Roman Tolchenov

comment:11 Changed 7 years ago by Arturs Bekasovs

  • Status changed from verifying to reopened
  • Resolution fixed deleted

Found a problem with multi-period dead times.

comment:12 Changed 7 years ago by Arturs Bekasovs

  • Status changed from reopened to inprogress

Refs #8597. Fix problem with multi-period dead times.

Changeset: 9974373e79bf32743e9417dda33639d14868a65c

comment:13 Changed 7 years ago by Arturs Bekasovs

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:14 Changed 7 years ago by Roman Tolchenov

  • Status changed from verify to closed

Merge remote-tracking branch 'origin/feature/8597_muon_load_algorithm'

Full changeset: d5fca1ce0382546d36a4b45ae9315da4be37a693

comment:15 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9441

Note: See TracTickets for help on using tickets.