Ticket #7638 (closed: invalid)

Opened 7 years ago

Last modified 5 years ago

Workaround for log nperiods for SANS2D

Reported by: Gesner Passos Owned by: Gesner Passos
Priority: major Milestone: Release 3.0
Component: SANS Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

Loading SANS2D event files there is no nperiods log information. This cause the reduction to fail when it asks to : 

ws.getRun().getLogData('nperiods').value

Make these calls dependend on the existence of nperiods, if it does not exists, assume 1 as default.

Change History

comment:1 Changed 7 years ago by Gesner Passos

  • Status changed from new to inprogress

comment:2 Changed 7 years ago by Gesner Passos

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

Sorry,

where the getLogData('nperios') is called it is already protected. We could do better asking first if the property is present. But, it does work in the same way.

comment:3 Changed 7 years ago by Russell Taylor

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

comment:4 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to closed

Yes, both the places this call appears are inside a try-except.

comment:5 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8483

Note: See TracTickets for help on using tickets.