Ticket #9417 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

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:1 Changed 6 years ago by Anders Markvardsen

  • Status changed from new to assigned

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:

  1. Open MuonAnalysis interface in Mantidplot
  1. select CHRONUS as instrument and click on the 'current run' button and you should see a plot
  1. 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:

  1. Open MuonAnalysis interface in Mantidplot
  1. select CHRONUS as instrument and click on the 'current run' button and you should see a plot
  1. 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

Note: See TracTickets for help on using tickets.