Ticket #6214 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Muon: bug when invalid det ID specified

Reported by: Anders Markvardsen Owned by: Anders Markvardsen
Priority: critical Milestone: Release 2.4
Component: MantidPlot Keywords:
Cc: Blocked By:
Blocking: Tester: Karl Palmen

Description

Steve writes:

I’ve noticed that if you’re editing a list of detectors in the grouping tab and mistakenly insert a nonexistent detector (perhaps by omitting a comma in the list) an error box is correctly generated and plotting is disabled. However, correcting the list doesn’t appear to restore plotting ... it seems to be necessary to restart Mantid to sort this out,

Change History

comment:1 Changed 8 years ago by Anders Markvardsen

Fix: Muon: bug when invalid det ID specified. re #6214

Changeset: 52a293784e9a91be8527823b2c0b4253f17af863

comment:2 Changed 8 years ago by Anders Markvardsen

  • Status changed from new to accepted

To test

  1. Open MuonAnalysis interface
  1. Hit browse button and open e.g. any of the MUSR datasets in AutoTestData
  1. click on the grouping option tab
  1. Edit one of the field in the 'Detector IDs' column and add a none existing ID like 10000 (all muon instruments has less than this number of detectors)
  1. the effect of 4 should be that the 'plot' button is greyed out
  1. Now delete the number added in 4 and if the 'plot' button becomes clickable again than OK

comment:3 Changed 8 years ago by Anders Markvardsen

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

comment:4 Changed 8 years ago by Anders Markvardsen

Fix: Muon: bug when invalid det ID specified. re #6214

Changeset: 52a293784e9a91be8527823b2c0b4253f17af863

comment:5 Changed 8 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:6 Changed 8 years ago by Karl Palmen

  • Status changed from verifying to closed

After doing 4 a waning message box appears and then plot is dissabled. I can then change the grouping back to something else.

I also not there is nothing to stop overlapping groups from being defined.

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7060

Note: See TracTickets for help on using tickets.