Ticket #7364 (closed: wontfix)

Opened 7 years ago

Last modified 5 years ago

ConjoinWorkspaces does not work with EventWorskpace properly

Reported by: Wenduo Zhou Owned by: Wenduo Zhou
Priority: major Milestone: Release 2.6
Component: Framework Keywords:
Cc: zhouw@… Blocked By: #7362
Blocking: Tester: Mathieu Doucet

Description (last modified by Mathieu Doucet) (diff)

Download the attached python script problem.py. It should work for the purpose to conjoin 2 workspaces. Spectrum with workspace index 0 is the high resolution TOF data. While the other one is the low resolution data.

If you comment out the 2 ConvertToMatrixWorkspace algorithms in the script, i.e., ConjoinWorkspaces will work on 2 event workspaces, you will find the final workspace to have 2 identical spectra.

Also, add another property 'spectrum number offset'. Such that the spectrum number of the first spectrum from 2nd workspaces may not be a consecutive number to the spectrum number of spectra of original first workspace.

Attachments

problem.py (1.0 KB) - added by Wenduo Zhou 7 years ago.

Change History

Changed 7 years ago by Wenduo Zhou

comment:1 Changed 7 years ago by Peter Peterson

Actually if you just set CheckOverlapping=False the error goes away. However, there is still a bug with it not reasonably changing the spectra numbers in this case.

comment:2 Changed 7 years ago by Wenduo Zhou

  • Description modified (diff)

comment:3 Changed 7 years ago by Wenduo Zhou

  • Status changed from new to accepted
  • Owner changed from Peter Peterson to Wenduo Zhou

comment:4 Changed 7 years ago by Wenduo Zhou

Russell told me the history of this algorithm. I've found that the best solution is to use AppendSpectra instead but not modify ConjoinWorkspaces for ticket #7336.

comment:5 Changed 7 years ago by Wenduo Zhou

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

comment:6 Changed 7 years ago by Mathieu Doucet

  • Status changed from verify to verifying
  • Description modified (diff)
  • Tester set to Mathieu Doucet

comment:7 Changed 7 years ago by Mathieu Doucet

  • Status changed from verifying to closed

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8210

Note: See TracTickets for help on using tickets.