search

art-framework-suite / art

The implementation of the art physics event processing framework
Other
2 stars 7 forks source link

Request additional output from EventIDFilter #71

Open knoepfel opened 3 years ago

knoepfel commented 3 years ago

This issue has been migrated from https://cdcvs.fnal.gov/redmine/issues/16706 (FNAL account required) Originally created by @kutschke on 2017-05-30 15:16:51

Thanks for providing EventIDFilter ( issue 13860 ).

I request that you add some additional functionality:

  1. At the end of the job, print out the number of events selected from each range.
  2. If an event is found twice, throw an exception

About 1). My main interest is if I find zero events in some range, which likely indicates a typo in configuration.

About 2), I can live with this only be implemented for ranges that are a single event wide, not a true range of events. But it would be great if the general solution were available; the limitations of our data handling systems are such that it is not likely one will be asking for more than a few hundreds of thousands of events per job - so I am not worried about about excessive time or memory for most use cases.

My preference is that all safety features be on by default and that people be able to disable them if they feel that the performance penalty is unacceptable.

knoepfel commented 3 years ago

Comment by @knoepfel on 2017-06-12 16:40:51

The first feature you request is feasible, and that is the one we will accept and provide. The second feature is duplicated by Redmine issue 16354, which has already been accepted.