Ticket #5172 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Indirect - New Scripts From Spencer

Reported by: Peter Parker Owned by: Peter Parker
Priority: blocker Milestone: Release 2.1
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description

Spencer would like to take the upcoming release as an opportunity to package his new Python scripts with Mantid, with an eye for them to be road-tested before they are plugged into various parts of the Indirect interfaces.

Change History

comment:1 Changed 8 years ago by Peter Parker

Refs #5172 - Adding scripts.

Changeset: 99d18a64e20fe2c8649bc899a7a968d25e746194

comment:2 Changed 8 years ago by Peter Parker

  • Priority changed from major to blocker

Changing to blocker to make some modifications.

comment:3 Changed 8 years ago by Peter Parker

Refs #5172 - Changes to Scripts

Changeset: fe31d9eb8030a7a7a464b9dca5b1085a1085fb9d

comment:4 Changed 8 years ago by Peter Parker

Refs #5172 - Temp fix for linux64 warnings.

Taking out Linux64 import statements until we can get hold of the relevant .so files.

Changeset: 431d40e8c1aadc735b1f791ba057bc179f02c79c

comment:5 Changed 8 years ago by Peter Parker

Refs #5172 - Adding linux 64 .so files.

Changeset: 6827b8adde49a5695bd51c35c8d1a2d3e28a9d7a

comment:6 Changed 8 years ago by Peter Parker

Revert "Refs #5172 - Temp fix for linux64 warnings."

This reverts commit 431d40e8c1aadc735b1f791ba057bc179f02c79c, which is no longer needed now the .so files have been committed.

Changeset: 2ab125204cb63d9250c2cbf7b0fb94d8f2686533

comment:7 Changed 8 years ago by Peter Parker

Refs #5172 - Final clean up of IndirectBayes.py from Spencer.

Changeset: f9a63b970f4e40abccd531e62d6e4a53a2169991

comment:8 Changed 8 years ago by Peter Parker

Revert "Revert "Refs #5172 - Temp fix for linux64 warnings.""

This reverts commit 2ab125204cb63d9250c2cbf7b0fb94d8f2686533, putting back in place the removal of code that imports the .so files.

Changeset: 7d39053c8bbe68db9cb9f2f905bc808026a58d22

comment:9 Changed 8 years ago by Peter Parker

Revert "Refs #5172 - Adding linux 64 .so files."

This reverts commit 6827b8adde49a5695bd51c35c8d1a2d3e28a9d7a, which is the addition of linux .so files.

Changeset: 512bb3f914ab472e6ccafea998ef5f884672e43b

comment:10 Changed 8 years ago by Peter Parker

  • Status changed from new to accepted

comment:11 Changed 8 years ago by Peter Parker

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

Closing. This ticket contains reverts of reverts to do with linux64 libraries and as such is a little hard to follow. To test:

  • Reinstall Mantid. Please use today's 32bit Windows RC.
  • Check for the existance of the following files:
    • MantidInstall/plugins/PythonAlgs/
      • ForCE.py
      • Jump.py
      • QLines.py
      • Quest.py
      • ResNorm.py
    • MantidInstall/scripts/Inelastic/
      • IndirectBayes.py
      • IndirectForce.py
      • IndirectTrans.py
      • CEFit_win32.pyd
      • QLdata_win32.pyd
      • QLres_win32.pyd
      • QLse_win32.pyd
      • Quest_win32.pyd
      • ResNorm_win32.pyd
      • SSfit_win32.pyd
      • erange_win32.pyd
  • Start up Mantid and make sure there are no errors/warnings relating to these files posted to the results log.
  • Repeat with any other version of the RC (Windows 64-bit is fine).

Note:

The very purpose of this ticket is to package scripts with this release so that they may be tested by users, and so testing of the scripts themselves to verify this ticket is uneccesary.

I have already raised #5231 to sort out errant messages posted during system tests.

Last edited 8 years ago by Peter Parker (previous) (diff)

comment:12 Changed 8 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:13 Changed 8 years ago by Martyn Gigg

  • Status changed from verifying to closed

I can confirm that the scripts are present and starting both the linux & windows variants produces no error messages.

comment:14 Changed 8 years ago by Peter Parker

Refs #5172 - Changes to Scripts

Changeset: fe31d9eb8030a7a7a464b9dca5b1085a1085fb9d

comment:15 Changed 8 years ago by Peter Parker

Refs #5172 - Adding linux 64 .so files.

Changeset: 6827b8adde49a5695bd51c35c8d1a2d3e28a9d7a

comment:16 Changed 8 years ago by Peter Parker

Revert "Refs #5172 - Temp fix for linux64 warnings."

This reverts commit 431d40e8c1aadc735b1f791ba057bc179f02c79c, which is no longer needed now the .so files have been committed.

Changeset: 2ab125204cb63d9250c2cbf7b0fb94d8f2686533

comment:17 Changed 8 years ago by Peter Parker

Refs #5172 - Final clean up of IndirectBayes.py from Spencer.

Changeset: f9a63b970f4e40abccd531e62d6e4a53a2169991

comment:18 Changed 8 years ago by Peter Parker

Revert "Revert "Refs #5172 - Temp fix for linux64 warnings.""

This reverts commit 2ab125204cb63d9250c2cbf7b0fb94d8f2686533, putting back in place the removal of code that imports the .so files.

Changeset: 7d39053c8bbe68db9cb9f2f905bc808026a58d22

comment:19 Changed 8 years ago by Peter Parker

Revert "Refs #5172 - Adding linux 64 .so files."

This reverts commit 6827b8adde49a5695bd51c35c8d1a2d3e28a9d7a, which is the addition of linux .so files.

Changeset: 512bb3f914ab472e6ccafea998ef5f884672e43b

comment:20 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6018

Note: See TracTickets for help on using tickets.