Ticket #11359 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Promote MD CoordinateSystem to member in MD workspaces

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 3.4
Component: Framework Keywords:
Cc: Blocked By:
Blocking: #11350 Tester: Owen Arnold

Description

Currently, the MD workspace stores the value for its coordinate system in a log on the first experiment info within the workspace. This field really applies to the workspace as a whole and not a particular run so it should be a separate field in the class and saved file.

This was uncovered in #11350 and will avoid having to load an experiment info when it is file backed just to set a single field.

Change History

comment:1 Changed 6 years ago by Martyn Gigg

Owen noted that we should also do this for a PeaksWorkspace.

comment:2 Changed 6 years ago by Martyn Gigg

  • Status changed from new to inprogress

Store the MD coordinate system in its own field.

Refs #11359

Changeset: 181db647a8435f2bec56975402a0c769c5c60376

comment:3 Changed 6 years ago by Martyn Gigg

Load and save the MD coordinate system

Backwards compatability is preserved by checking for the log as it would have existed. Refs #11359

Changeset: e053866e88944edcd36cc75d3beb5d3327633a93

comment:4 Changed 6 years ago by Martyn Gigg

Formatting changes before the real changes.

Refs #11359

Changeset: c97d33c7a122b7125c799a10a09be0145d1fba9e

comment:5 Changed 6 years ago by Martyn Gigg

Load and save coordinate field in PeaksWorkspace.

It is still backwards compatible with the old log-entry format. Refs #11359

Changeset: cee29dc758aa3c8b3ff93e438e352f917eb52f09

comment:6 Changed 6 years ago by Martyn Gigg

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

This is being verified as pull request #407.

comment:7 Changed 6 years ago by Martyn Gigg

Copy coordinate system to new workspaces

This used to be done by copying the experiment info but is handled separately now that the coordinate is an extra field. Refs #11359

Changeset: d78fd4db26c1b70f046dda58b33e5ceaab2ffdcb

comment:8 Changed 6 years ago by Martyn Gigg

W Matrix has been removed as it was unused.

Refs #11359

Changeset: 489eedc8f866a696a7a091879aad436e70ae79b6

comment:9 Changed 6 years ago by Owen Arnold

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

comment:10 Changed 6 years ago by Owen Arnold

jenkins retest this please

comment:11 Changed 6 years ago by Martyn Gigg

Include HDF5 for MDAlgorithmsTest

On Windows they are in a non-standard location so must be explicitly included. Refs #11359

Changeset: 82ce4e6a6dcdabe3e4bd5318ef1278b106ec5abf

comment:12 Changed 6 years ago by Martyn Gigg

Jenkins, retest this please

comment:13 Changed 6 years ago by Martyn Gigg

Jenkins, retest this please

comment:14 Changed 6 years ago by Martyn Gigg

Jenkins, retest this please

comment:15 Changed 6 years ago by Owen Arnold

  • Status changed from verifying to closed

Merge pull request #407 from mantidproject/11359_md_coordinate_as_own_field

MD coordinate as separate field

Full changeset: 574efd1fe23cc214c201dada4126d59a3a3ae0df

comment:16 Changed 6 years ago by Owen Arnold

Code looks good, and I've tested this for both Peaks and MDWorkspaces.

comment:17 Changed 6 years ago by Martyn Gigg

Merge branch '11359_md_coordinate_as_own_field' into 11350_converttomd_no_expt_info_loading

Conflicts:

Code/Mantid/Framework/MDEvents/src/MDEventWorkspace.cpp

Refs #11350

Full changeset: bacb8b42e87bb737fc99ad844b9011fd729f98db

comment:18 Changed 5 years ago by Nick Draper

Somehow these slipped through without a resolution. Set to Fixed.

comment:19 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 12198

Note: See TracTickets for help on using tickets.