Ticket #7179 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

In result tab look for run, temperaure and magnetic_field

Reported by: Anders Markvardsen Owned by: Arturs Bekasovs
Priority: major Milestone: Release 3.1
Component: Muon Keywords:
Cc: francis.pratt@… Blocked By: #7178
Blocking: Tester: Peter Parker

Description (last modified by Arturs Bekasovs) (diff)

In Results Table tab in the Values window in addition to log value entries also allow user to select the following entries if present in the Run object:

  • temperature
  • magnetic_field

Display these at the top.

Change History

comment:1 Changed 7 years ago by Anders Markvardsen

  • Cc francis.pratt@… added

comment:2 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.6 to Backlog

Moved to backlog at the code freeze for R2.6

comment:3 Changed 7 years ago by Anders Markvardsen

  • Owner changed from Anders Markvardsen to Arturs Bekasovs
  • Description modified (diff)

comment:4 Changed 7 years ago by Arturs Bekasovs

  • Milestone changed from Backlog to Release 3.1

comment:5 Changed 7 years ago by Arturs Bekasovs

  • Summary changed from MUON: In result tab look for run, temperaure and magnetic_field to [Muon] In result tab look for run, temperaure and magnetic_field

comment:6 Changed 7 years ago by Arturs Bekasovs

  • Summary changed from [Muon] In result tab look for run, temperaure and magnetic_field to In result tab look for run, temperaure and magnetic_field

comment:7 Changed 7 years ago by Arturs Bekasovs

  • Component changed from MantidPlot to Muon
  • Description modified (diff)

comment:8 Changed 7 years ago by Arturs Bekasovs

  • Status changed from new to inprogress

Refs #7179. Refactor log table population function.

Better naming, using set instead of endlessly iterating through lists, removed redundant checks for whether we have enough rows in the table.

Changeset: 835779ce8e05520f5bc30533f572393af024c573

comment:9 Changed 7 years ago by Arturs Bekasovs

Refs #7179. More refactoring.

Changeset: 9edded2870af5bd8d202843c27dfa0c4d8698239

comment:10 Changed 7 years ago by Arturs Bekasovs

Refs #7179. Add code for displaying non-timeseries log values.

Changeset: b3f41a6099af89735db127022ee4fd9aa1756b55

comment:11 Changed 7 years ago by Arturs Bekasovs

Refs #7179. Remove run number log specific code.

Changeset: 07ca919eb052188becd4d9cbe4b8928050dca259

comment:12 Changed 7 years ago by Arturs Bekasovs

Refs #7179. Make a list of non-timeseries logs a static const.

Changeset: e9097a7d44fd6b3dfc1c06967824b8f8d7d6732f

comment:13 Changed 7 years ago by Arturs Bekasovs

Refs #7179. Sort logs before displaying them.

Changeset: 7b16a86d79060b510f89f73445826b97539eb23e

comment:14 Changed 7 years ago by Arturs Bekasovs

Tester:

A simple test case:

  1. In MuonAnalysis, load AutoTestData/MUSR00015189.nxs.
  2. Do a sequential fit for 15189-15199.
  3. Go to the Results tab and switch to view sequential fittings for the label you've used. The following three non-timeseries log values should be at the top:
    • run_number
    • sample_magn_field
    • sample_temp
  4. run_number should be selected by default. Select sample_temp, sample_magn_field and Field_Danfysik in addition to that. Make sure the values are right:
    • run_number should correspond to the one selected, i.e. 15189-15199
    • sample_temp should be stable at 290 for all the runs
    • sample_magn_field should be increasing from 1350 to 1450
    • Field_Danfysik values should be close to the ones of sample_magn_field.
  5. Now return to the home page and load AutoTestData/argus0026287.nxs.
  6. Do a single fit.
  7. Go to the Results tab. ARGUS files do not have neither sample_magn_field nor sample_temp, so they shouldn't be in the list. The run_number should be present in all the Muon Nexus files though.
Last edited 7 years ago by Arturs Bekasovs (previous) (diff)

comment:15 Changed 7 years ago by Arturs Bekasovs

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

comment:16 Changed 7 years ago by Peter Parker

  • Status changed from verify to verifying
  • Tester set to Peter Parker

comment:17 Changed 7 years ago by Peter Parker

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/7179_muon_result_table_field_and_temp'

Full changeset: ​9d5d2c5ffc50a6b2ea6faa50492642139c075825

comment:18 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8025

Note: See TracTickets for help on using tickets.