Skip to content

Commit

Permalink
Depend on *all* DQMTokens.
Browse files Browse the repository at this point in the history
This is stupid and wrong, but actually works.
  • Loading branch information
schneiml committed Nov 7, 2019
1 parent 29c77f3 commit 7add316
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions DQM/SiStripMonitorClient/plugins/SiStripOfflineDQM.cc
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,11 @@ SiStripOfflineDQM::SiStripOfflineDQM(edm::ParameterSet const& pSet)

// explicit dependency to make sure the QTest reults needed here are present
// already in endRun.
consumes<DQMToken, edm::InRun>(edm::InputTag("siStripQTester"));
consumes<DQMToken, edm::InLumi>(edm::InputTag("siStripQTester"));
// TODO: this should only read the one Token needed, not all. Otherwise we'd
// get a cyclic dependency if this module produces DQMTokens itself. This
// would happen e.g. if this was a DQMEDHarvester, as it should be.
consumesMany<DQMToken, edm::InRun>();
consumesMany<DQMToken, edm::InLumi>();
usesResource("DQMStore");
}

Expand Down

0 comments on commit 7add316

Please sign in to comment.