Ticket #5069 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Create workspace for polygon rebinning

Reported by: Michael Reuter Owned by: Michael Reuter
Priority: major Milestone: Release 2.1
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description

A new workspace needs to be created for handling the output of the polygon rebinning since there is the need to track the fraction overlap areas.

Change History

comment:1 Changed 9 years ago by Michael Reuter

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Michael Reuter

Refs #5069. Adding start of rebinning output workspace.

Changeset: 94149559214b997bf1291218084f0fe9f3826232

comment:3 Changed 8 years ago by Michael Reuter

Refs #5069 and #5070. Updates to workspace and algorithm.

The rebinning workspace now has the fractional area tracking. This has been propogated to the necessary places in the algorithm for handling.

Changeset: 69407694d84f8682ba9aa26950d317a816e0bb47

comment:4 Changed 8 years ago by Michael Reuter

Refs #5069 and #5070. Updates to workspace and algorithm.

The rebinning workspace now has the fractional area tracking. This has been propogated to the necessary places in the algorithm for handling.

Changeset: 69407694d84f8682ba9aa26950d317a816e0bb47

comment:5 Changed 8 years ago by Michael Reuter

Refs #5069. Workspace can write to a processed NeXus file.

Changeset: b850567ed3cf38250a62fd2581ec78b0a7fbb9ac

comment:6 Changed 8 years ago by Michael Reuter

Refs #5069. Adding a setter and debugging info.

Changeset: f21e39f1ca10ba4c559bef817570433459ac6627

comment:7 Changed 8 years ago by Michael Reuter

Refs #5069. Adding test helper for workspace.

I've had to short-circuit the tests until I can figure out why I can't actually use the helper I've created.

Changeset: 2a0947ebe7c3224ed7651444a1463a7575f1927c

comment:8 Changed 8 years ago by Michael Reuter

Refs #5069. Fixed helper and uncommented tests.

Changeset: 96c557c07c6d9df3fa288c1095cad3e6749204fc

comment:9 Changed 8 years ago by Michael Reuter

Refs #5069. Fixing helper and testing more things on workspace.

Changeset: fdca34f74e8e79d41be84bf272f177b57a886634

comment:10 Changed 8 years ago by Michael Reuter

Refs #5069. Workspace can write to a processed NeXus file.

Changeset: b850567ed3cf38250a62fd2581ec78b0a7fbb9ac

comment:11 Changed 8 years ago by Michael Reuter

Refs #5069. Adding a setter and debugging info.

Changeset: f21e39f1ca10ba4c559bef817570433459ac6627

comment:12 Changed 8 years ago by Michael Reuter

Refs #5069. Adding test helper for workspace.

I've had to short-circuit the tests until I can figure out why I can't actually use the helper I've created.

Changeset: 2a0947ebe7c3224ed7651444a1463a7575f1927c

comment:13 Changed 8 years ago by Michael Reuter

Refs #5069. Fixed helper and uncommented tests.

Changeset: 96c557c07c6d9df3fa288c1095cad3e6749204fc

comment:14 Changed 8 years ago by Michael Reuter

Refs #5069. Fixing helper and testing more things on workspace.

Changeset: fdca34f74e8e79d41be84bf272f177b57a886634

comment:15 Changed 8 years ago by Michael Reuter

Refs #5069. Making SumSpectra work for RebinnedOutput workspace.

Changeset: aafd18430482b4e98746fe1341ff94ced6f9a5d7

comment:16 Changed 8 years ago by Michael Reuter

Refs #5069. Removing debugging statement.

Changeset: 4721447cabcaea1428eb81fce31f8fcf7dbe078b

comment:17 Changed 8 years ago by Michael Reuter

Refs #5069. Can now load a RebinnedOutput NeXus file.

Changeset: ac00f6b9fafe04a27152b05bb2163ba3f0db74a7

comment:18 Changed 8 years ago by Michael Reuter

Refs #5069. Transpose nearly works. Nans go missing though.

Changeset: 7187e927755f6fbbaa22284325cc1e80919c856f

comment:19 Changed 8 years ago by Michael Reuter

Refs #5069. Can use the workspace from Rebin2D.

An optional flag was added to use the fractional area tracking workspace. A RebinnedOutput workspace will returned from the algorithm.

Changeset: 794db01a142655267be2b43a1ac9905d0d507d41

comment:20 Changed 8 years ago by Michael Reuter

Refs #5069. Adding more Transpose tests.

Changeset: 067aa316a69abbc9efdca07a4a1d8ae1307462c8

comment:21 Changed 8 years ago by Michael Reuter

Refs #5069. Integration now works.

Changeset: 4449be70654f6a27e331d4f234c7bc1a8807064b

comment:22 Changed 8 years ago by Michael Reuter

Refs #5069. Try to make Windows happy.

Changeset: 57f85e0910628bf279ff4f0ec891930b1a931170

comment:23 Changed 8 years ago by Michael Reuter

Refs #5069. Correcting Integration implementation.

The old implementation performed the undo frac area, sum and redo frac area when calculating. It should actually be just a straight sum for Integration.

Changeset: 7214811f328c488805cec55c7ba75f544b213596

comment:24 Changed 8 years ago by Michael Reuter

Refs #5069. Streamlined Integration implementation.

Changeset: 6f20c5a66288d61209fca722e30cec8ce2780667

comment:25 Changed 8 years ago by Michael Reuter

Refs #5069. Don't let algorithm modify input data.

Changeset: 65cb783967901afd2419b74d15b40cef52e0f194

comment:26 Changed 8 years ago by Michael Reuter

Refs #5069. Making SumSpectra work for RebinnedOutput workspace.

Changeset: aafd18430482b4e98746fe1341ff94ced6f9a5d7

comment:27 Changed 8 years ago by Michael Reuter

Refs #5069. Removing debugging statement.

Changeset: 4721447cabcaea1428eb81fce31f8fcf7dbe078b

comment:28 Changed 8 years ago by Michael Reuter

Refs #5069. Can now load a RebinnedOutput NeXus file.

Changeset: ac00f6b9fafe04a27152b05bb2163ba3f0db74a7

comment:29 Changed 8 years ago by Michael Reuter

Refs #5069. Transpose nearly works. Nans go missing though.

Changeset: 7187e927755f6fbbaa22284325cc1e80919c856f

comment:30 Changed 8 years ago by Michael Reuter

Refs #5069. Can use the workspace from Rebin2D.

An optional flag was added to use the fractional area tracking workspace. A RebinnedOutput workspace will returned from the algorithm.

Changeset: 794db01a142655267be2b43a1ac9905d0d507d41

comment:31 Changed 8 years ago by Michael Reuter

Refs #5069. Adding more Transpose tests.

Changeset: 067aa316a69abbc9efdca07a4a1d8ae1307462c8

comment:32 Changed 8 years ago by Michael Reuter

Refs #5069. Integration now works.

Changeset: 4449be70654f6a27e331d4f234c7bc1a8807064b

comment:33 Changed 8 years ago by Michael Reuter

Refs #5069. Try to make Windows happy.

Changeset: 57f85e0910628bf279ff4f0ec891930b1a931170

comment:34 Changed 8 years ago by Michael Reuter

Refs #5069. Correcting Integration implementation.

The old implementation performed the undo frac area, sum and redo frac area when calculating. It should actually be just a straight sum for Integration.

Changeset: 7214811f328c488805cec55c7ba75f544b213596

comment:35 Changed 8 years ago by Michael Reuter

Refs #5069. Streamlined Integration implementation.

Changeset: 6f20c5a66288d61209fca722e30cec8ce2780667

comment:36 Changed 8 years ago by Michael Reuter

Refs #5069. Don't let algorithm modify input data.

Changeset: 65cb783967901afd2419b74d15b40cef52e0f194

comment:37 Changed 8 years ago by Michael Reuter

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

The best way of testing this it use the work in #5070 to generate on of these workspaces. Only a couple of algorithms have been modified to use this workspace, but it would be interesting to know what other ones need to be handled.

comment:38 Changed 8 years ago by Michael Reuter

  • Status changed from verify to reopened
  • Resolution fixed deleted

comment:39 Changed 8 years ago by Michael Reuter

  • Status changed from reopened to accepted

comment:40 Changed 8 years ago by Michael Reuter

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

Think this works for Rebin2D, but not sure about the other algorithms. That may need to wait.

comment:41 Changed 8 years ago by Martyn Gigg

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

comment:42 Changed 8 years ago by Martyn Gigg

  • Status changed from verifying to closed

Confirmed that this works for SofQW3 and Rebin2D as the answers look sensible so it must be doing it's job properly.

comment:43 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5915

Note: See TracTickets for help on using tickets.