Ticket #5763 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Clean up banked event packet parsing

Reported by: Ross Miller Owned by: Ross Miller
Priority: major Milestone: Release 2.3
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Stuart Campbell

Description

The parser code for the ADARA banked events - specifically the event iterator functions - contains a fair amount of redundant code and also doesn't handle a few odd corner cases.

Change History

comment:1 Changed 8 years ago by Ross Miller

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Ross Miller

Treat pixels from bank 0xFFFFFFFE as error pixels

Refs #5763

Changeset: c920aa62b3145ada58729026fc72949d2e688e66

comment:3 Changed 8 years ago by Ross Miller

Clean up event iterator functions in BankedEventPkt

The event iterator functions (firstEvent() & nextEvent()) are a little easier to understand and handle certain corner cases better (such as packets with multiple sources, but some sources having no events).

Refs #5763

Changeset: b45af3f2e1d093838dc1e74c362ee96d80e34a8d

comment:4 Changed 8 years ago by Ross Miller

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

comment:5 Changed 8 years ago by Ross Miller

Treat pixels from bank 0xFFFFFFFE as error pixels

Refs #5763

Changeset: c920aa62b3145ada58729026fc72949d2e688e66

comment:6 Changed 8 years ago by Ross Miller

Clean up event iterator functions in BankedEventPkt

The event iterator functions (firstEvent() & nextEvent()) are a little easier to understand and handle certain corner cases better (such as packets with multiple sources, but some sources having no events).

Refs #5763

Changeset: b45af3f2e1d093838dc1e74c362ee96d80e34a8d

comment:7 Changed 8 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:8 Changed 8 years ago by Stuart Campbell

  • Status changed from verifying to closed

tested on HYSPEC

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6609

Note: See TracTickets for help on using tickets.