Ticket #8391 (closed: fixed)
[IBayes] Replace erange Fortran code with Python
Reported by: | Samuel Jackson | Owned by: | Samuel Jackson |
---|---|---|---|
Priority: | major | Milestone: | Release 3.1 |
Component: | Indirect Inelastic | Keywords: | |
Cc: | spencer.howells@… | Blocked By: | |
Blocking: | Tester: | Roman Tolchenov |
Description
Spencer has kindly provided a python implementation of the erange function used in IndirectBayes. Meaning one less f2py call.
Attachments
Change History
Changed 7 years ago by Samuel Jackson
- Attachment IndirectBayes.py added
comment:1 Changed 7 years ago by Samuel Jackson
- Status changed from new to inprogress
Refs #8391 Added python implementation of energy range function
Changeset: b675835654e420480ebeaae916ee980865568016
comment:2 Changed 7 years ago by Samuel Jackson
Refs #8391 Correcting function to pass system tests.
Changeset: 7e6fd151f57e4dfba6f149c05e402f41cb148903
comment:3 Changed 7 years ago by Samuel Jackson
Refs #8391 Remove unused Fortran modules.
As the implementation in this ticket replaces them.
Changeset: 3890f85f841365bbbdc3f554eca4108354d54501
comment:4 Changed 7 years ago by Samuel Jackson
- Status changed from inprogress to verify
- Resolution set to fixed
To Tester
This function is shared between most of the Bayes programs (ResNorm, Stretch and Quasi). The system test should already be checking that we're getting the same output as before, but I would also suggest running through them with some example data. Instructions for running them can be found here:
http://www.mantidproject.org/Indirect:Indirect_Bayes
I've also removed the Fortran module erange, so make sure there are no references to it left in the project.
comment:6 Changed 7 years ago by Samuel Jackson
- Status changed from verify to reopened
- Resolution fixed deleted
comment:7 Changed 7 years ago by Samuel Jackson
- Status changed from reopened to inprogress
Refs #8391 Fix bin width counting.
Changeset: 5c5d04fec16c39f36b22f9c36bbbce32a92245cb
comment:8 Changed 7 years ago by Samuel Jackson
Refs #8391 Fix bin widths again.
Changeset: 53f6c8ae2caa0ce677acfdaaa5fc0fbcc5ef5763
comment:9 Changed 7 years ago by Samuel Jackson
Refs #8391 Fix bound overflow issue.
Changeset: 441f2e29230b3fcce38a115966f167c2e5e9f55d
comment:10 Changed 7 years ago by Samuel Jackson
Merge branch 'feature/8391_bayes_erange_python' into develop
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectBayes.py
Refs #8391
Changeset: c51367a183d01fec16632851b7f6b63c37830e4b
comment:11 Changed 7 years ago by Samuel Jackson
Refs #8391 Updated reference results.
Changeset: 5db0afc2814c25fea141360d38efedd8629cb42a
comment:12 Changed 7 years ago by Samuel Jackson
- Status changed from inprogress to verify
- Resolution set to fixed
To Tester
As above. Additionally, check the system tests are passing. Also be sure to run with the upper bound of the energy range set to the maximum possible and check that Mantid does not crash.
comment:13 Changed 7 years ago by Roman Tolchenov
- Status changed from verify to verifying
- Tester set to Roman Tolchenov
comment:14 Changed 7 years ago by Samuel Jackson
Merge branch 'feature/8391_bayes_erange_python' into develop
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectBayes.py
Refs #8391
Changeset: c51367a183d01fec16632851b7f6b63c37830e4b
comment:15 Changed 7 years ago by Samuel Jackson
- Status changed from verifying to closed
Merge branch 'feature/8391_bayes_erange_python' of github.com:mantidproject/mantid into feature/8391_bayes_erange_python
Full changeset: 82658b19ac9fc7936d5b37af7007836f5d2aef1d
comment:16 Changed 7 years ago by Roman Tolchenov
Merge remote-tracking branch 'origin/feature/8391_bayes_erange_python'
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectBayes.py
Re #8391
Changeset: 11ec9b43a89b14e5921bd485096a2c1f33393aa9
comment:17 Changed 7 years ago by Roman Tolchenov
Merge remote-tracking branch 'origin/feature/8391_bayes_erange_python'
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectBayes.py
Re #8391
Full changeset: 11ec9b43a89b14e5921bd485096a2c1f33393aa9
comment:18 Changed 7 years ago by Roman Tolchenov
Merge branch 'master' into develop
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectBayes.py
Re #8391
Changeset: 71e8a9ca65361a0627a2dbe94b5b8adb029ba44f
comment:19 Changed 7 years ago by Martyn Gigg
It looks as if there was a branch in the systemtests repository that also needs merging. The tests on master on windows are currently failing.
comment:20 Changed 7 years ago by Roman Tolchenov
Merge remote-tracking branch 'origin/feature/8391_bayes_erange_python'
Full changeset: 5d879c186badfb22b880045fccce30e9b1a68472
comment:21 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 9236
Updated version of the script