Ticket #9643 (closed: fixed)
Create an algorithm to load VULCAN's calibration file
Reported by: | Wenduo Zhou | Owned by: | Wenduo Zhou |
---|---|---|---|
Priority: | major | Milestone: | Release 3.3 |
Component: | Framework | Keywords: | |
Cc: | petersonpf@… | Blocked By: | |
Blocking: | Tester: | Vickie Lynch |
Description (last modified by Wenduo Zhou) (diff)
VULCAN has its own calibration file to do time focusing. The idea is somehow similar as that in Mantid, but it works in TOF but not d-spacing. Write a new algorithm similar to LoadCalFile() to load VULCAN's offset file, DIFC file and bad pixel file.
A discrepancy of the naming of banks between Mantid and current VUCLAN's IDL code is found. Modify the geometry file to make it consistent.
Attachments
Change History
comment:1 Changed 6 years ago by Wenduo Zhou
- Status changed from new to infoneeded
- Milestone changed from Backlog to Release 3.3
comment:5 Changed 6 years ago by Wenduo Zhou
Added partially finished new algorithm. Refs #9643.
Changeset: 5d9aa2c2da34036ae66a8be1565e0c5980e83a80
comment:7 Changed 6 years ago by Wenduo Zhou
Refs #9643. Implemented the conversion from VULCAN offset to Mantid.
Also, the geometry file of VULCAN is slightly modified to make the definition of bank numbers consistent between Mantid and that VULCAN uses now.
A unit test data file is added, too.
Changeset: cb1617640d1286b88e1fa38c758a70aac89d94ec
comment:8 Changed 6 years ago by Wenduo Zhou
Fixed some issues during completing unit tests. Refs #9643.
Changeset: 4059ed68df5c78c0c1f493f7eb1bf71183134d68
comment:9 Changed 6 years ago by Wenduo Zhou
Refs #9643. Fixed errors and warnings.
Including Windows, Mac, Doxygen and CppCheck.
Changeset: a5cd27ee384068039ccbd877e4cf22f4c035303b
comment:10 Changed 6 years ago by Wenduo Zhou
Enabled to import mask file. Refs #9643.
Also
- Add unit test file;
- Write a new unit test for it;
Changeset: 7031e9329538bc6e8449ee15c0d219040fdfd169
comment:11 Changed 6 years ago by Wenduo Zhou
Added partially finished new algorithm. Refs #9643.
Changeset: 02d725ad08fa9e73b902c69a346fdc8045bf3735
comment:12 Changed 6 years ago by Wenduo Zhou
Refs #9643. Implemented the conversion from VULCAN offset to Mantid.
Also, the geometry file of VULCAN is slightly modified to make the definition of bank numbers consistent between Mantid and that VULCAN uses now.
A unit test data file is added, too.
Changeset: d0b66ba57a41dbd5295b2db5cb0d3e1bb94a614f
comment:13 Changed 6 years ago by Wenduo Zhou
Fixed some issues during completing unit tests. Refs #9643.
Changeset: ef36d1d39b1b8ca95dc2859dcc62d10e29ac282e
comment:14 Changed 6 years ago by Wenduo Zhou
Refs #9643. Fixed errors and warnings.
Including Windows, Mac, Doxygen and CppCheck.
Changeset: 4f067ab68d03724cfc746fe5a427c049bdd446ae
comment:15 Changed 6 years ago by Wenduo Zhou
Enabled to import mask file. Refs #9643.
Also
- Add unit test file;
- Write a new unit test for it;
Changeset: 04076ebc49fc77f5b7f1f139c3ef95d3766feb6b
comment:16 Changed 6 years ago by Wenduo Zhou
Fixed some issues. Refs #9643.
Including
- sphinx documentation due to change of geometry file;
- added unit test file again due to rebasing with master;
- cppcheck warning.
Changeset: 872315475803fc927c6c20491ae4f4713446caf9
comment:17 Changed 6 years ago by Wenduo Zhou
Refs #9643. Fixed conflicts and others.
Conflicts:
Test/AutoTestData/pid_offset_vulcan_new.dat
And
- Rebased by master;
- Fixed cppcheck warnings;
- Fixed a sphinx doc error due to change in definition of bank ID.
Changeset: 8d324c2873c3fcc87fb42476771c468fbe354e03
comment:18 Changed 6 years ago by Wenduo Zhou
Refs #9643. Forgot to add to commit.
Changeset: 0a3564192a789cebdb9943d876a4d45527efdb47
comment:19 Changed 6 years ago by Wenduo Zhou
Fixed a git issue with unit test file. Refs #9643.
Changeset: ec43af5c112c971328fb557f087607ba718dccaa
comment:20 Changed 6 years ago by Wenduo Zhou
Refs #9643. Tried to fix windows warning.
Changeset: 4306a1c84f943a9f71a8830cde856ba59a9f9ca9
comment:21 Changed 6 years ago by Wenduo Zhou
Refs #9643. Added doc file.
Changeset: 20a92da9a8428e1670ec4fb6ad39275ae2789a8d
comment:22 Changed 6 years ago by Wenduo Zhou
Refs #9643. Fixed doc errors.
Changeset: 7627f22f7315f074b5502239107507e53552678b
comment:23 Changed 6 years ago by Martyn Gigg
Fix index error in vector access.
This was discovered by the windows debug build. Refs #9643
Changeset: ac1eafbdb19d87a1e016f8fffa84ff16e01247d5
comment:24 Changed 6 years ago by Wenduo Zhou
Added one more output workspace. Refs #9643.
Changeset: 3c045eabebee1dbcd5c0a2fa9c8811d221c14731
comment:25 Changed 6 years ago by Wenduo Zhou
- Status changed from inprogress to verify
- Resolution set to fixed
comment:26 Changed 6 years ago by Wenduo Zhou
Refs #9643. Updated unit test.
(1) Unit test is updated to remove the extra output workspace.
(2) Debugging output is cleaned up.
Changeset: 7f07600b88d00cc89c0c255a899c5acc7376a44e
comment:27 Changed 6 years ago by Vickie Lynch
- Status changed from verify to verifying
- Tester set to Vickie Lynch
comment:28 Changed 6 years ago by Vickie Lynch
Refs #9643 fix conflict
Changeset: cef03e948a85952bcbe3ed144464f81677b6e9da
comment:29 Changed 6 years ago by Vickie Lynch
- Status changed from verifying to closed
LoadDspacemap should be removed in another ticket since this algorithm does the same thing with more options.
comment:30 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 10486