Ticket #5013 (closed: fixed)
AbragamTest fails on ISIS Snowleapard
| Reported by: | Karl Palmen | Owned by: | Russell Taylor |
|---|---|---|---|
| Priority: | major | Milestone: | Release 2.2 |
| Component: | Mantid | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Peter Parker |
Description
The unit test for the Abragam fitting function added for ticket #4956 failed on ISIS Snowleopard and so had to be disabled. It works on other systems (except perhaps ORNL Snowleopard).
Change History
comment:2 Changed 8 years ago by Owen Arnold
Error in previous message. Should have gone against ticket #5103.
comment:3 Changed 8 years ago by Owen Arnold
refs #5013. Move PV Plugins.
Move PVPLUGINS to a subdirectory of PVPLUGINS. This has two advantages. Firstly it maintains the possiblity to keep the loading of PV related mantid components separate from those in Mantid Plugins. Secondly it means that Paraview standalone does not crash when PV_PLUGIN_PATH is used. Only Paraview plugins sit in the subdirectory, so the Paraview loader doesnt get confused by the presence of these other libraries
Changeset: 6e6ffbc431abb06baa413c29f3f6d6ae4a834fe2
comment:4 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.1 to Release 2.2
Moved at end of release 2.1
comment:5 Changed 8 years ago by Nick Draper
- Owner set to Russell Taylor
- Status changed from new to assigned
Russell, does this happen at ORNL? If so could you help Karl out.
comment:7 Changed 8 years ago by Russell Taylor
Re #5013. Re-introduce body of AbragamTest.
It should be fine on all OSX builds now, following upgrade of Intel compiler on ISIS buildserver (see #5461).
Changeset: 53f24740d9c3884c41c2e6139f89d187d5a36d22
comment:8 Changed 8 years ago by Russell Taylor
- Status changed from accepted to verify
- Resolution set to fixed
The test passes now.
comment:9 Changed 8 years ago by Russell Taylor
Re #5013. Re-introduce body of AbragamTest.
It should be fine on all OSX builds now, following upgrade of Intel compiler on ISIS buildserver (see #5461).
Changeset: 53f24740d9c3884c41c2e6139f89d187d5a36d22
comment:10 Changed 8 years ago by Russell Taylor
Re #5013. Re-introduce body of AbragamTest.
It should be fine on all OSX builds now, following upgrade of Intel compiler on ISIS buildserver (see #5461).
Changeset: 53f24740d9c3884c41c2e6139f89d187d5a36d22
comment:11 Changed 8 years ago by Russell Taylor
Re #5013. Re-introduce body of AbragamTest.
It should be fine on all OSX builds now, following upgrade of Intel compiler on ISIS buildserver (see #5461).
Changeset: 53f24740d9c3884c41c2e6139f89d187d5a36d22
comment:12 Changed 8 years ago by Peter Parker
- Status changed from verify to verifying
- Tester set to Peter Parker
comment:13 Changed 8 years ago by Peter Parker
- Status changed from verifying to closed
CurveFittingTest_AbragamTest does indeed pass at ORNL and ISIS on Snow Leopard.
comment:14 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 5859
refs #5013. Move PV Plugins.
Move PVPLUGINS to a subdirectory of PVPLUGINS. This has two advantages. Firstly it maintains the possiblity to keep the loading of PV related mantid components separate from those in Mantid Plugins. Secondly it means that Paraview standalone does not crash when PV_PLUGIN_PATH is used. Only Paraview plugins sit in the subdirectory, so the Paraview loader doesnt get confused by the presence of these other libraries