Ticket #5546 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

POLREF Periods not filtering

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: blocker Milestone: Release 2.2
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description (last modified by Owen Arnold) (diff)

See attached script. This script this isn't performing the filtering expected! Fails when loading raw POLREF data, like POLREF00006224.raw from cycle_12_1. The number of values returned before and after filtering is the same. This is because the periods are not being loaded correctly.

the icpevent log provides a CHANGE_PERIOD flag rather than a CHANGE PERIOD flag.

Attachments

filterlogs.py (330 bytes) - added by Owen Arnold 8 years ago.

Change History

Changed 8 years ago by Owen Arnold

comment:1 Changed 8 years ago by Owen Arnold

  • Status changed from new to accepted
  • Description modified (diff)

comment:2 Changed 8 years ago by Owen Arnold

  • type changed from enhancement to defect

comment:3 Changed 8 years ago by Owen Arnold

refs #5546 Should work for both cases now.

Changeset: 5e281bd17ac4fc8c4954cd24a527312f1f322a1c

comment:4 Changed 8 years ago by Owen Arnold

Tester: Attached script should now work according to the ticket description.

comment:5 Changed 8 years ago by Owen Arnold

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

comment:6 Changed 8 years ago by Owen Arnold

refs #5546. Fix MSBuild warning

Changeset: 5dbea00ee6b3eeea23c1bc13c5f44e25b7c9b658

comment:7 Changed 8 years ago by Owen Arnold

refs #5546 Should work for both cases now.

Changeset: 5e281bd17ac4fc8c4954cd24a527312f1f322a1c

comment:8 Changed 8 years ago by Owen Arnold

refs #5546. Fix MSBuild warning

Changeset: 5dbea00ee6b3eeea23c1bc13c5f44e25b7c9b658

comment:9 Changed 8 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:10 Changed 8 years ago by Martyn Gigg

  • Status changed from verifying to closed

The event log must be parsed correctly now as the period logs are correct.

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6392

Note: See TracTickets for help on using tickets.