Ticket #3677 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Support VATES Simple GUI in Embedded Mode

Reported by: Owen Arnold Owned by: Michael Reuter
Priority: critical Milestone: Iteration 30
Component: Mantid Keywords:
Cc: reuterma.ornl.gov Blocked By:
Blocking: Tester: Owen Arnold

Description

Need to be able to load workspaces out of the ADS rather than from files. Need to provide source plugins to support this. GUI frameworks will have to handle the loading of the plugins as well as passing a single "WSName" parameter. These plugins are only relevant for the embedded mode of usage, so should not be made available for the stand-alone usage.

Change History

comment:1 Changed 9 years ago by Owen Arnold

In [14463]:

refs #3677 #3662. 3677 will probably involve adapting/extending the MVP presenters to handle loading from the ADS rather than files. Needed to fix template usage anyway.

comment:2 Changed 9 years ago by Owen Arnold

In [14464]:

refs #3677 #3662. Strip includes out of header files now that not templated.

comment:3 Changed 9 years ago by Owen Arnold

In [14465]:

refs #3677 #3662. Fix warning.

comment:4 Changed 9 years ago by Owen Arnold

In [14486]:

refs #3677. MVP presenter that works from the ADS via WorkspaceProvider and expects IMDEventWorkspaces.

comment:5 Changed 9 years ago by Owen Arnold

In [14487]:

refs #3677. Unused argument warning fix.

comment:6 Changed 9 years ago by Owen Arnold

In [14489]:

refs #3677. PV Source plugin which extracts IMDEventWorkspaces out of the ADS.

comment:7 Changed 9 years ago by Owen Arnold

In [14490]:

refs #3677. Fix unused arg warnings and give better docs.

comment:8 Changed 9 years ago by Michael Reuter

In [14529]:

Using correct workspace type for ADSWorkspaceProvider. This refs #3677.

comment:9 Changed 9 years ago by Owen Arnold

In [14584]:

refs #3677. Add info only property. Add support for getGeometryXML in loading presenter. Test.

comment:10 Changed 9 years ago by Owen Arnold

In [14585]:

refs #3677. Try to fix unused warning in test header.

comment:11 Changed 9 years ago by Owen Arnold

  • Status changed from new to accepted

comment:12 Changed 9 years ago by Owen Arnold

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

comment:13 Changed 9 years ago by Michael Reuter

  • Status changed from verify to reopened
  • Resolution fixed deleted

comment:14 Changed 9 years ago by Michael Reuter

  • Owner changed from Owen Arnold to Michael Reuter
  • Status changed from reopened to accepted

comment:15 Changed 9 years ago by Michael Reuter

In [14647]:

Needed to expose the TimestepValues property. This refs #3677.

comment:16 Changed 9 years ago by Michael Reuter

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

Now I think it's done.

comment:17 Changed 9 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:18 Changed 9 years ago by Owen Arnold

  • Status changed from verifying to closed

Agreed. This does work.

We currently have a few minor issues with deployment in different environments, but these are being processed under separate tickets and should appear as fixes in the current iteration.

comment:19 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4524

Note: See TracTickets for help on using tickets.