Ticket #10568 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

SortXAxis produces an additional workspace

Reported by: Dan Nixon Owned by: Dan Nixon
Priority: major Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Federico M Pouzols

Description

Running the following script should result in two workspaces; out and out_sorted. 

CreateSampleWorkspace(OutputWorkspace='out')
SortXAxis(InputWorkspace='out', OutputWorkspace='out_sorted')

A third workspace, outws, is also created.

Change History

comment:1 Changed 6 years ago by Dan Nixon

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Dan Nixon

  • Status changed from assigned to inprogress

Tidy up SortXAxis, remove additional workspace left in ADS

Refs #10568

Changeset: 52c8665a7a63aa7c900fb0f8188297ab46ba95cc

comment:3 Changed 6 years ago by Dan Nixon

To test:

  • Run the script in the description on master, notice the additional workspace
  • Do the same with the fix applied, notice only the output workspaces from CreateSampleWorkspace and SortXAxis are listed

comment:4 Changed 6 years ago by Dan Nixon

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

comment:5 Changed 6 years ago by Federico M Pouzols

  • Status changed from verify to verifying
  • Tester set to Federico M Pouzols

comment:6 Changed 6 years ago by Federico Montesino Pouzols

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/10568_sort_x_axis_produces_additional_workspaces'

Full changeset: ​3d5bf932a6d8bcb60f7dc2e93d54289e0fbd8186

comment:7 Changed 6 years ago by Federico M Pouzols

This works. Before, I'd get a third/tmp workspace called outws. After the changes I only get 'out' and 'out_sorted'. Code changes look fine and units tests pass.

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​11410

Note: See TracTickets for help on using tickets.