Ticket #9720 (closed: fixed)
Apply Corrections fails with old OSIRIS files
Reported by: | Samuel Jackson | Owned by: | Samuel Jackson |
---|---|---|---|
Priority: | critical | Milestone: | Release 3.2 |
Component: | Indirect Inelastic | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Peter Parker |
Description
Reported by Sanghamitra:
Today morning I tried to use ‘Use Can’ option available in IDA Apply Correction on Run 110041 (sample) and 103358 (can) (OSIRIS, Analyser: graphite, 002, mapping:All), but receive the following error:
Error in execution of algorithm Minus: X arrays must match when performing this operation on a 2D workspaces. ValueError: X arrays must match when performing this operation on a 2D workspaces. at line 16 in '<Interface>' caused by line 1302 in 'C:/MantidInstall/scripts/Inelastic\IndirectDataAnalysis.py' caused by line 567 in 'C:/MantidInstall/bin\mantid\simpleapi.py'
This is because 103358 is an old run with a shorter range. Her suggestion is to rebin the can to the sample if the bins don't match and spit out a warning that it is doing so.
Change History
comment:1 Changed 6 years ago by Samuel Jackson
- Summary changed from Apply Corrections fails with old versions OSIRIS files to Apply Corrections fails with old OSIRIS files
comment:2 Changed 6 years ago by Samuel Jackson
- Priority changed from major to critical
- Milestone changed from Backlog to Release 3.2
Modifying status on Nick's advice.
comment:4 Changed 6 years ago by Samuel Jackson
- Status changed from assigned to inprogress
Refs #9720 Check if the binning is the same.
If not run RebinToWorkspace and output a warning.
Changeset: 74b5ff449674289fbb9f46cf86d2801a47e0facf
comment:5 Changed 6 years ago by Samuel Jackson
Refs #9720 Add option to rebin can to match sample.
Changeset: 9f0e804c18c5c94b0ab05cd8dd4cd1aeff7f5cec
comment:6 Changed 6 years ago by Samuel Jackson
Refs #9720 Modify interface to ask user to confirm operation.
Changeset: 4ac912e79d9ac8a4488d8957b73330c7cddc698b
comment:7 Changed 6 years ago by Samuel Jackson
Merge branch 'bugfix/9720_apply_corrections_fails_with_older_files' into develop
Refs #9720
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectDataAnalysis.py
Changeset: d124bb6a982b7787ba0df562e07beb5829529504
comment:8 Changed 6 years ago by Samuel Jackson
Refs #9720 Include option in complex corrections.
Changeset: b0c34d12efa1ae3c3a8a91f37743b7d24b5d4521
comment:9 Changed 6 years ago by Samuel Jackson
Merge branch 'bugfix/9720_apply_corrections_fails_with_older_files' into develop
Refs #9720
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectDataAnalysis.py
Changeset: 44024bd59739813fedc7b0c2ed8ca58c12305127
comment:10 Changed 6 years ago by Samuel Jackson
Refs #9720 Better error message.
Changeset: a9e53966121169d65bb927f39bea88d075795654
comment:11 Changed 6 years ago by Samuel Jackson
To Test
- Open the manage directories window and check search instrument archive.
- Open Convert to energy and select OSIRIS as the instrument.
- Type the run numbers 110041, 103358 and click run.
- Open apply corrections and use 110041 as the sample and 103358 as the can.
- Running should ask you if you wish to rebin one to the other.
- Clicking yes should allow the routine to succeed.
- Clicking no should stop the program.
- Code review.
comment:12 Changed 6 years ago by Samuel Jackson
- Status changed from inprogress to verify
- Resolution set to fixed
comment:13 Changed 6 years ago by Peter Parker
- Status changed from verify to verifying
- Tester set to Peter Parker
comment:14 Changed 6 years ago by Peter Parker
- Status changed from verifying to reopened
- Resolution fixed deleted
Could I pass this one back to you so that you can remerge master into it, please? There was a conflict in IndirectDataAnalysis.py at around lines 967 and 989.
comment:15 Changed 6 years ago by Samuel Jackson
- Status changed from reopened to inprogress
Merge remote-tracking branch 'origin/master' into bugfix/9720_apply_corrections_fails_with_older_files
Refs #9720
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectDataAnalysis.py
Changeset: 3d7f6697947863040bfea7204c570c11aec44a24
comment:16 Changed 6 years ago by Samuel Jackson
Merge branch 'bugfix/9720_apply_corrections_fails_with_older_files' into develop
Refs #9720
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectDataAnalysis.py
Changeset: 454a87399f9335cbfe6f5ed552b308438ab35b9c
comment:17 Changed 6 years ago by Samuel Jackson
- Status changed from inprogress to verify
- Resolution set to fixed
comment:19 Changed 6 years ago by Peter Parker
- Status changed from verifying to reopened
- Resolution fixed deleted
As discussed, a slight improvement could be made to the message, making it clear that nothing at all will happen if the user clicks "no" (should this be "cancel"?), and that clicking "yes" (should this be "ok"?) will do the rebinning, though it is not recommended a user does this.
Apart from that, the rest is great.
comment:20 Changed 6 years ago by Samuel Jackson
- Status changed from reopened to inprogress
Merge branch 'bugfix/9720_apply_corrections_fails_with_older_files' into develop
Refs #9720
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectDataAnalysis.py
Changeset: d124bb6a982b7787ba0df562e07beb5829529504
comment:21 Changed 6 years ago by Samuel Jackson
Merge branch 'bugfix/9720_apply_corrections_fails_with_older_files' into develop
Refs #9720
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectDataAnalysis.py
Changeset: 44024bd59739813fedc7b0c2ed8ca58c12305127
comment:22 Changed 6 years ago by Samuel Jackson
Merge branch 'bugfix/9720_apply_corrections_fails_with_older_files' into develop
Refs #9720
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectDataAnalysis.py
Changeset: 454a87399f9335cbfe6f5ed552b308438ab35b9c
comment:23 Changed 6 years ago by Samuel Jackson
- Status changed from inprogress to verify
- Resolution set to fixed
comment:24 Changed 6 years ago by Peter Parker
- Status changed from verify to reopened
- Resolution fixed deleted
Reopened to address problem where we can't re-run the correction routine.
comment:26 Changed 6 years ago by Samuel Jackson
Refs #9720 Restructure GUI code to prevent it accidentally quitting.
Changeset: 5c70b7581d0039b9f197e588a91fe0f40fd83461
comment:27 Changed 6 years ago by Samuel Jackson
Refs #9720 Fix typo.
Changeset: efda758b8945c1809a17bd7501c1dda2ede47987
comment:28 Changed 6 years ago by Samuel Jackson
- Status changed from inprogress to verify
- Resolution set to fixed
comment:30 Changed 6 years ago by Peter Parker
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/bugfix/9720_apply_corrections_fails_with_older_files'
Full changeset: 28aa2f03e027e4351b93cacdfda37e2347f487cc
comment:31 Changed 6 years ago by Peter Parker
I think we're there on this one.
comment:32 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 10562