Ticket #2760 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Add search for instrument across facilities to configservice

Reported by: Stuart Campbell Owned by: Stuart Campbell
Priority: major Milestone: Iteration 28
Component: Mantid Keywords:
Cc: doucetm@… Blocked By:
Blocking: Tester: Anders Markvardsen

Description


Change History

comment:1 Changed 10 years ago by Stuart Campbell

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Mathieu Doucet

  • Cc doucetm@… added

comment:3 Changed 10 years ago by Russell Taylor

(In [10553]) Fix system tests to make currently necessary facility adjustment. Re #2760.

comment:4 Changed 10 years ago by Stuart Campbell

(In [10667]) Moving implementation out of header and into source file. refs #2760

comment:5 Changed 10 years ago by Stuart Campbell

(In [10669]) Add some debugging info. refs #2760

comment:6 Changed 10 years ago by Stuart Campbell

(In [10670]) Added getInstrument() to ConfigService instance. refs #2760

comment:7 Changed 10 years ago by Stuart Campbell

(In [10674]) Going to move default facility tests to ConfigServiceTests as they are part of ConfigService. First step - remove from FacilitiesTest. refs #2760

comment:8 Changed 10 years ago by Stuart Campbell

(In [10697]) In ConfigService, renamed Facility() --> getFacility() Added a setFacility() refs #2760

comment:9 Changed 10 years ago by Stuart Campbell

(In [10698]) Remove all the facility setting nonsense from the DataHandling tests. refs #2760

comment:10 Changed 10 years ago by Stuart Campbell

(In [10699]) Remove all the facility setting nonsense from the Nexus tests. refs #2760

comment:11 Changed 10 years ago by Stuart Campbell

(In [10701]) Remove all the facility setting nonsense from the Algorithms tests. refs #2760

comment:12 Changed 9 years ago by Stuart Campbell

(In [10757]) makeFileName to look for the instrument across facilities. refs #2760

comment:13 Changed 9 years ago by Stuart Campbell

(In [10758]) Let's try removing some more of the facility setting... refs #2760

comment:14 Changed 9 years ago by Stuart Campbell

(In [10759]) Shouldn't need to reset the facility. refs #2760

comment:15 Changed 9 years ago by Stuart Campbell

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

comment:16 Changed 9 years ago by Stuart Campbell

(In [10840]) Changed to use getInstrument on ConfigService rather than through facility. refs #2760

comment:17 Changed 9 years ago by Stuart Campbell

(In [10841]) Changed to use setFacility rather than setting the property by name. refs #2760

comment:18 Changed 9 years ago by Stuart Campbell

(In [10842]) Changed to use setFacility rather than setting the property by name. refs #2760

comment:19 Changed 9 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:20 Changed 9 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Cleaned up the handling of different facilities and searching for instruments

comment:21 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​3607

Note: See TracTickets for help on using tickets.