Ticket #6758 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Implement DataService for external data associated with the SCD Reduction GUI.

Reported by: Ruth Mikkelson Owned by: Ruth Mikkelson
Priority: major Milestone: Release 2.5
Component: Mantid Keywords:
Cc: mikkelsond@… Blocked By:
Blocking: Tester: Russell Taylor

Description

The SCD Reduction GUI should interact with other GUI's/applications that involve associated data. This Singleton should be very much like the AnalysisDataServiceImplement DataService for external data associated with the SCD Reduction GUI.

Change History

comment:1 Changed 8 years ago by Ruth Mikkelson

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Ruth Mikkelson

Refs #6758 Inital Checkin for this ticket

Changeset: 9ac3fbfbce2222f0c3454342befe00e88991a98a

comment:3 Changed 8 years ago by Ruth Mikkelson

Refs #6758 Inital Checkin for this ticket

Changeset: 9ac3fbfbce2222f0c3454342befe00e88991a98a

comment:4 Changed 8 years ago by Ruth Mikkelson

  • Cc mikkelsond@… added
  • Status changed from accepted to verify
  • Resolution set to fixed

This is a VERY basic DataService. Utility methods will be added in further tickets once users start using this( which will be soon)

comment:5 Changed 8 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:6 Changed 8 years ago by Russell Taylor

A few comments on this:

  • There was a plan, albeit one that appears not to be written down anywhere, for the AnalysisDataService to be changed so that it is able to store more than just Workspaces. This was the reason why the Workspace class gained the DataItem base class. However, the job was never finished. This kind of change is best done right after a release, so I'll create a ticket for myself to do it then. At that point I'd like this class to be removed and for you to switch to using that.
  • It would be better for the semantics to have UDTs stored in here rather than just vectors. In fact, that will be a requirement later as you'll have to inherit from DataItem.
  • The name of the class could cause confusion as Notifications in Mantid are something different. I'll let it go for now if this class is to be temporary.

comment:7 Changed 8 years ago by Ruth Mikkelson

Refs #6758 Inital Checkin for this ticket

Changeset: 9ac3fbfbce2222f0c3454342befe00e88991a98a

comment:8 Changed 8 years ago by Russell Taylor

  • Status changed from verifying to closed

I've let this in for now, but it needs to be strictly temporary. I've created a blocker ticket #6805 for removing this class and using the AnalysisDataService instead once that's been modified appropriately.

comment:9 Changed 8 years ago by Ruth Mikkelson

Refs #6758 Inital Checkin for this ticket

Changeset: 9ac3fbfbce2222f0c3454342befe00e88991a98a

comment:10 Changed 7 years ago by Ruth Mikkelson

Refs #6758 Inital Checkin for this ticket

Changeset: 9ac3fbfbce2222f0c3454342befe00e88991a98a

comment:11 Changed 7 years ago by Ruth Mikkelson

Refs #6758 Inital Checkin for this ticket

Changeset: 9ac3fbfbce2222f0c3454342befe00e88991a98a

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7604

Note: See TracTickets for help on using tickets.