Ticket #5160 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

CheckWorkspaceMatch should work for chunked and non-chunked powder reduction results

Reported by: Vickie Lynch Owned by: Vickie Lynch
Priority: major Milestone: Release 2.1
Component: Mantid Keywords:
Cc: petersonpf@… Blocked By:
Blocking: Tester: Shelly Ren

Description

CompressEvents for chunks of a bank causes the results to be different. Change to chunking by banks using the number of events in each bank to determine how many banks are in each chunk. Then do not CompressWorkspaces for the results of DiffractionFocussing until all chunks are summed. Move the save for pdfgetn until after all the chunks are summed.

Change History

comment:1 Changed 8 years ago by Vickie Lynch

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Vickie Lynch

Refs #5160 change chunking to by bank

Changeset: bc8b937476a97fc6f990a8dcb3ee3fcdfb643eee

comment:3 Changed 8 years ago by Vickie Lynch

Refs #5160 rename variables

Changeset: 30d4c6e7b19ab3b5dd60a94aa9f634e4674022ec

comment:4 Changed 8 years ago by Vickie Lynch

Refs #5160 last chunk had too many banks

Changeset: 01970ee4d20ffdaca4959547b0e5955c58e9e70a

comment:5 Changed 8 years ago by Vickie Lynch

Refs #5160 at least one bank in every chunk

Changeset: 7c1b382b976800e067de9939b8580fc7c5635d10

comment:6 Changed 8 years ago by Vickie Lynch

Refs #5160 check for more chunks than banks

Changeset: 08199098fb470e2ca15989a54b8b46bc45abc3df

comment:7 Changed 8 years ago by Vickie Lynch

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

To test run SNSPowderReduction with and without chunking and CheckWorkspacesMatch on results.

comment:8 Changed 8 years ago by Vickie Lynch

Refs #5160 change chunking to by bank

Changeset: bc8b937476a97fc6f990a8dcb3ee3fcdfb643eee

comment:9 Changed 8 years ago by Vickie Lynch

Refs #5160 rename variables

Changeset: 30d4c6e7b19ab3b5dd60a94aa9f634e4674022ec

comment:10 Changed 8 years ago by Vickie Lynch

Refs #5160 last chunk had too many banks

Changeset: 01970ee4d20ffdaca4959547b0e5955c58e9e70a

comment:11 Changed 8 years ago by Vickie Lynch

Refs #5160 at least one bank in every chunk

Changeset: 7c1b382b976800e067de9939b8580fc7c5635d10

comment:12 Changed 8 years ago by Vickie Lynch

Refs #5160 check for more chunks than banks

Changeset: 08199098fb470e2ca15989a54b8b46bc45abc3df

comment:13 Changed 8 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:14 Changed 8 years ago by Michael Reuter

  • Tester changed from Michael Reuter to Shelly Ren

comment:15 Changed 8 years ago by Shelly Ren

This ticket is tested in #5104 and #5134 for chunking and pdfget.

comment:16 Changed 8 years ago by Shelly Ren

  • Status changed from verifying to closed

comment:17 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6006

Note: See TracTickets for help on using tickets.