Ticket #7262 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Change log msg to warning about invalid event_index field in LoadEventNexus

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.6
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Gesner Passos

Description

Also update the message to say that filtering on this data is not possible at all.

The current msg is:

alg->getLogger().information() << this->entry_name << "'s field 'event_index' seem to be invalid (> than the number of events in the bank). Filtering by time ignored.\n";

at line 584.

Change History

comment:1 Changed 7 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 7 years ago by Martyn Gigg

Due to other changes in LoadEventNexus the line number is now 566

comment:3 Changed 7 years ago by Martyn Gigg

Put msg about bad frame indexes at warning rather than information.

Also update them message to say filtering by time is not possible. Refs #7262

Changeset: 4ee2663543718ec12573c9e0b41a2f449c6d30b9

comment:4 Changed 7 years ago by Martyn Gigg

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

Branch: feature/7262_no_time_filter_event_nexus_msg

Tester: The example file we had is huge and so used for testing. There should be some SEQUOIA files around run 7872 that show the problem. If you can't get hold a file then a code inspection will have to do.

comment:5 Changed 7 years ago by Gesner Passos

  • Status changed from verify to verifying
  • Tester set to Gesner Passos

comment:6 Changed 7 years ago by Gesner Passos

  • Status changed from verifying to closed

The changing of the level is effectively done.

comment:7 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8108

Note: See TracTickets for help on using tickets.