Ticket #11469 (closed: fixed)
Bugfix: LoadMD does not load MDEventData correctly if initial splitting was applied
Reported by: | Anton Piccardo-Selg | Owned by: | Anton Piccardo-Selg |
---|---|---|---|
Priority: | major | Milestone: | Release 3.4 |
Component: | Framework | Keywords: | VATES |
Cc: | Blocked By: | ||
Blocking: | Tester: | Owen Arnold |
Description (last modified by Anton Piccardo-Selg) (diff)
Context
As part of the VATES AA an option to produce a forced initial splitting was added to the ConvertToMD algorithm. This produces a workspace which has a forced splitting in its first level of 50x50x50x50.
This is implemented by resetting the SplitInto option of the box controller to our desired values, letting the first box split and setting the SplitInto option back to its original value.
Issue
When the MDEventWorkspace is saved via SaveMD this knowledge of the initial splitting is lost, but it stores the original SplitInto option of the box controller. When the LoadMD algorithm is called to reload the data the expected size of the MDGridBox, which is read out from the SplitInto option and the actual size of the MDGridBox are not matched, which can cause an out of bounds error.
This means, that workspaces which have had a forced initial split applied to them cannot be saved and reloaded in a proper way.
Attachments
Change History
comment:1 Changed 6 years ago by Anton Piccardo-Selg
- Owner changed from Anton Piccardo-Selg to Nick Draper
- Status changed from new to assigned
comment:2 Changed 6 years ago by Anton Piccardo-Selg
- Owner changed from Nick Draper to Anton Piccardo-Selg
comment:5 Changed 6 years ago by Anton Piccardo-Selg
- Status changed from assigned to inprogress
Refs #11469 Add top level split storage to box controller
Changeset: c57e397bfc9f6775f9845411f86e258844b525af
comment:6 Changed 6 years ago by Anton Piccardo-Selg
Refs #11469 Fix for loading initial split data vsi
Changeset: 0d4fe7dce395bb1b1d7c6286801ca9dde7a5d927
comment:7 Changed 6 years ago by Anton Piccardo-Selg
Refs #11469 Added unit tests to box controller
Changeset: 73075b64a1f8bacbfc1ced58d0294d2861067248
comment:8 Changed 6 years ago by Anton Piccardo-Selg
Refs #11469 Resolving merge issue in box controller
Changeset: d3f847a15b1b206021c90f8d4fad3774215ad622
comment:9 Changed 6 years ago by Anton Piccardo-Selg
- Status changed from inprogress to verify
- Resolution set to fixed
This is being verified as pull request #532.
comment:10 Changed 6 years ago by Anton Piccardo-Selg
Refs #11469 Fix cpp check issue
Changeset: e500bf3fc14de3830dfb0fc0bc9114d01e3697eb
comment:11 Changed 6 years ago by Anton Piccardo-Selg
Refs #11469 Fix compiler warnings
Changeset: 7c26ba46e3c44e81db01d4e531e88f12b7209da9
comment:12 Changed 6 years ago by Owen Arnold
- Status changed from verify to verifying
- Tester set to Owen Arnold
comment:13 Changed 6 years ago by Owen Arnold
- Status changed from verifying to closed
Merge pull request #532 from mantidproject/bugfix/11469_fix_loadMD_issue_from_initial_splitting
Fix load issue for MD workspaces which have top level splitting
Full changeset: 9b1c5dcdb0fd8aaf5556e2b2cc62a5467719054a
comment:14 Changed 5 years ago by Nick Draper
Somehow these slipped through without a resolution. Set to Fixed.
comment:15 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 12308