Ticket #9417 (closed: fixed)
Enable 'current run' for chronus
Reported by: | Anders Markvardsen | Owned by: | Anders Markvardsen |
---|---|---|---|
Priority: | major | Milestone: | Release 3.2 |
Component: | Muon | Keywords: | |
Cc: | francis.pratt@… | Blocked By: | |
Blocking: | Tester: | Keith Brown |
Description
Change History
comment:2 Changed 6 years ago by Anders Markvardsen
- Status changed from assigned to inprogress
load currnet chronus run. re #9417
Changeset: 9c54fdb8e234b49e1392a3d759b12bef626c51fe
comment:3 Changed 6 years ago by Anders Markvardsen
- Status changed from inprogress to verify
- Resolution set to fixed
To test - ISIS ONLY:
- Open MuonAnalysis interface in Mantidplot
- select CHRONUS as instrument and click on the 'current run' button and you should see a plot
- check that it also works for e.g. the EMU instrument, by repeat 2 but for emu
comment:4 Changed 6 years ago by Jay Rainey
- Status changed from verify to verifying
- Tester set to Jay Rainey
comment:5 Changed 6 years ago by Jay Rainey
- Status changed from verifying to verify
- Tester Jay Rainey deleted
Re-opening for another tester as it's Windows specific.
comment:6 Changed 6 years ago by Owen Arnold
- Status changed from verify to verifying
- Tester set to Owen Arnold
comment:7 Changed 6 years ago by Owen Arnold
Yes, Jay has found a problem that I am experiencing too.
1) The interface assumes that you have access to the archive
2) The paths are windows specific.
I would suggest using the same code that Jay has been using to enable OS independent archive path creation. Happy to talk about other options here.
comment:8 Changed 6 years ago by Anders Markvardsen
- Status changed from verifying to reopened
- Resolution fixed deleted
OK this sounds like a general improvement.
I have reopen the ticket and we can talk about it next week
comment:9 Changed 6 years ago by Anders Markvardsen
A separate ticket has been created based on the comments above, http://trac.mantidproject.org/mantid/ticket/9486 .
As mentioned in this ticket: No Muon user have ever complained about this, hence interested they all appear to be windows users!
comment:10 Changed 6 years ago by Anders Markvardsen
- Status changed from reopened to verify
- Resolution set to fixed
To test - ISIS ONLY and WINDOWS user ONLY:
- Open MuonAnalysis interface in Mantidplot
- select CHRONUS as instrument and click on the 'current run' button and you should see a plot
- check that it also works for e.g. the EMU instrument, by repeat 2 but for emu
comment:11 Changed 6 years ago by Keith Brown
- Status changed from verify to verifying
- Tester changed from Owen Arnold to Keith Brown
comment:12 Changed 6 years ago by Keith Brown
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/9417_chronus_current_run'
Full changeset: 978236abf9ed3376284e4c783ac6a19828552c6b
comment:13 Changed 6 years ago by Keith Brown
Worked as described.
I did observe though that the rebinning parameters weren't cleared when switching instruments as the same invalid parameters were still there on my copy from my work on #9152 meaning it was tripping an error message
I'd ask the scientists if it would be ideal if when switching instruments, clearing the rebinning parameters for fixed and variable data binning would be advisable, as the old parameters may not be suitable for the new instrument
comment:14 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 10260