Ticket #9410 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Fix ZeroEdgePixels for ExpandSumAllPixels in SmoothNeighbours

Reported by: Vickie Lynch Owned by: Vickie Lynch
Priority: major Milestone: Release 3.2
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Wenduo Zhou

Description

ZeroEdgePixels does not work when using the ExpandSumAllPixels option.

Change History

comment:1 Changed 6 years ago by Vickie Lynch

Refs #9410 fix edge pixels

Changeset: 9c0cb353d5f0f6d47fc37d6ccfc12f271c909e22

comment:2 Changed 6 years ago by Vickie Lynch

  • Status changed from new to assigned

To test run script and then Show Instrument. Banks should have constant value with zeros 20 pixels from the edge. 

Load(Filename=r'TOPAZ_3132_event.nxs',OutputWorkspace='TOPAZ_3132_event')
SmoothNeighbours(InputWorkspace='TOPAZ_3132_event',OutputWorkspace='SumEdge20',SumPixelsX='256',SumPixelsY='256',ZeroEdgePixels='20',Radius='0.10000000000000001',SumNumberOfNeighbours='8',ExpandSumAllPixels='1')

comment:3 Changed 6 years ago by Vickie Lynch

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

comment:4 Changed 6 years ago by Vickie Lynch

  • Status changed from verify to reopened
  • Resolution fixed deleted

comment:5 Changed 6 years ago by Vickie Lynch

  • Status changed from reopened to inprogress

Refs #9410 temporary add of anvred corrections

Changeset: be241debcc42eef12829188468680176e45aec77

comment:6 Changed 6 years ago by Vickie Lynch

Refs #9410 remove temporary changes

Changeset: 998a7461fc1eecf85a962a117b4580a717423637

comment:7 Changed 6 years ago by Vickie Lynch

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

comment:8 Changed 6 years ago by Wenduo Zhou

  • Status changed from verify to verifying
  • Tester set to Wenduo Zhou

comment:9 Changed 6 years ago by Vickie Lynch

Refs #9410 add zero spectra for edge pixels

Changeset: ef6701408de5ce52bc80f5717feda188031fd74c

comment:10 Changed 6 years ago by Wenduo Zhou

I can see what should be. Ticket is passed.

comment:11 Changed 6 years ago by Wenduo Zhou

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/9410_edge_pixels_expand'

Full changeset: ​bec3daca835f905e8146a53617bb51baa4cc0a98

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​10253

Note: See TracTickets for help on using tickets.