Ticket #6772 (closed: wontfix)
Solve systemtest SANS2DMultiPeriodAddFiles
Reported by: | Gesner Passos | Owned by: | Gesner Passos |
---|---|---|---|
Priority: | minor | Milestone: | Release 3.2 |
Component: | Framework | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Peter Parker |
Description (last modified by Gesner Passos) (diff)
The SANS2DMultiPeriodAddFiles is failing in managed workspace. (Look for example the http://download.mantidproject.org/jenkins/job/is_test_winxp_master_managedws). You can reproduce the error by changing the Mantid.properties ManagedWorkspace.LowerMemoryLimit, and set it to 1
ManagedWorkspace.LowerMemoryLimit = 1
Then you will see the SANS2DMultiPeriodAddFiles system test to break. You can also just open mantid and try to load the file SANS2D00002500.nxs (systemtest/Data/SANS2D), than, right click on the workspace and Show Detectors. You will see that the detectors numbers are not loaded correctly.
The problem is very specific, it is related to managed workspaces and it seems to appear only on SANS2D data. Eventhough more recent SANS2D files does not show the same problem.
Still more curious, is that the commit that broke this system test was https://github.com/mantidproject/mantid/commit/60b2a49703b0bd2de719e1ea537dff99857080fe
Change History
comment:1 Changed 8 years ago by Wenduo Zhou
- Status changed from new to assigned
- Owner changed from Wenduo Zhou to Gesner Passos
comment:2 Changed 8 years ago by Russell Taylor
See also the more general ticket #6681, which is probably the same thing.
comment:6 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.6 to Backlog
Moved to the Backlog after R2.6
comment:7 Changed 7 years ago by Gesner Passos
- Priority changed from critical to minor
- Description modified (diff)
comment:10 Changed 7 years ago by Gesner Passos
- Status changed from new to verify
- Resolution set to wontfix
I think that there is no point to try to dig around this issue. We are not going to continue to support XP.
comment:12 Changed 7 years ago by Peter Parker
- Status changed from verify to verifying
- Tester set to Peter Parker
comment:13 Changed 7 years ago by Peter Parker
- Status changed from verifying to closed
It's looking like managed workspaces wont have a future, either.
comment:14 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7618