change type of frequency property

key4hep · Oct 28, 2024 · 85452c7 · 85452c7
1 parent 4400e0a
commit 85452c7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/k4FWCore/components/EventCounter.cpp b/k4FWCore/components/EventCounter.cpp
@@ -26,14 +26,14 @@ class EventCounter final : public Gaudi::Functional::Consumer<void(const EventCo
 
   void operator()(const EventContext& ctx) const override {
     ++m_count;
-    if ((m_frequency > 0) && (ctx.evt() % m_frequency == 0)) {
+    if (m_frequency && (ctx.evt() % m_frequency == 0)) {
       info() << "Processing event " << ctx.evt() << endmsg;
     }
   }
 
 private:
   mutable Gaudi::Accumulators::Counter<> m_count{this, "count"};
-  Gaudi::Property<int>                   m_frequency{this, "Frequency", 1, "How often to print the event number"};
+  Gaudi::Property<unsigned int>          m_frequency{this, "Frequency", 1, "How often to print the event number"};
 };
 
 DECLARE_COMPONENT(EventCounter)
diff --git a/test/k4FWCoreTest/options/TestEventCounter.py b/test/k4FWCoreTest/options/TestEventCounter.py
@@ -44,7 +44,7 @@
 
 scheduler = AvalancheSchedulerSvc(ThreadPoolSize=threads, ShowDataFlow=True, OutputLevel=WARNING)
 
-counter_1 = EventCounter("SilentEventCounter", Frequency=0, OutputLevel=INFO)
+counter_1 = EventCounter("SilentEventCounter", Frequency=-2, OutputLevel=INFO)
 counter_2 = EventCounter("EventCounter", Frequency=2, OutputLevel=INFO)