Ticket #6317 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

AlignAndFocusPowder with different input/output workspaces

Reported by: Peter Peterson Owned by: Wenduo Zhou
Priority: major Milestone: Release 2.4
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

The python code below does not work. However, changing the call to AlignAndFocusPowder to overwrite the input workspace is fine. 

PG3_11476 = LoadEventNexus(Filename="PG3_11476_event.nxs")
PG3_11476_foc = AlignAndFocusPowder(InputWorkspace=PG3_11476, 
             Params=[-0.0004],
             CalFileName="/SNS/PG3/2013_1_11A_CAL/PG3_PAC_d11475_2012_11_29.cal")

Change History

comment:1 Changed 8 years ago by Wenduo Zhou

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Wenduo Zhou

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

comment:3 Changed 8 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:4 Changed 8 years ago by Russell Taylor

  • Status changed from verifying to reopened
  • Resolution worksforme deleted

This doesn't work for me. The behaviour is still exactly as Pete describes. Tested on analysis.sns.gov using 2.3.1005.

comment:5 Changed 8 years ago by Wenduo Zhou

Fix the issue as reported. Refs #6317.

Changeset: dc5c948aa42bef15aba9c968bd1d670b39a380ba

comment:6 Changed 8 years ago by Wenduo Zhou

  • Status changed from reopened to accepted

comment:7 Changed 8 years ago by Wenduo Zhou

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

comment:8 Changed 8 years ago by Russell Taylor

  • Status changed from verify to verifying

comment:9 Changed 8 years ago by Russell Taylor

It works now, but note that the use of the setName method (m_outputW->setName(getProperty("OutputWorkspace"));) is highly dodgy. Only the ADS should be setting names on workspaces and the method will probably be going away under #6187.

comment:10 Changed 8 years ago by Russell Taylor

  • Status changed from verifying to closed

Forgot to check 'Fixed'...

comment:11 Changed 8 years ago by Wenduo Zhou

Fix the issue as reported. Refs #6317.

Changeset: dc5c948aa42bef15aba9c968bd1d670b39a380ba

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7163

Note: See TracTickets for help on using tickets.