Skip to content

Commit 0e0ede5

Browse files
committed
Honour the CCDB time machine for the mu(bc) distribution
The mu(bc) lookup in CollisionContextTool uses its own CCDBManagerInstance, which -- unlike BasicCCDBManager -- never picks up the time-machine constraint from ALICEO2_CCDB_CONDITION_NOT_AFTER. FT0/Calib/EventsPerBc was therefore resolved against the present even when a time machine was requested, quietly switching the sampler and changing the collision context at an unchanged seed. Bunch filling and mean vertex, which go through BasicCCDBManager, were pinned correctly. Now carry the constraint over explicitly. See O2-7093.
1 parent be5a2bb commit 0e0ede5

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

Steer/src/CollisionContextTool.cxx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -440,6 +440,10 @@ int main(int argc, char* argv[])
440440
// for now construct a specific CCDBManager for this query
441441
o2::ccdb::CCDBManagerInstance ccdb_inst(ccdb_info.server + std::string(":") + ccdb_info.port);
442442
ccdb_inst.setFatalWhenNull(false);
443+
// this is a private instance, so it does not inherit the time-machine
444+
// constraint that BasicCCDBManager picks up from the environment;
445+
// carry it over explicitly (a 0 here means "unconstrained" anyway)
446+
ccdb_inst.setCreatedNotAfter(o2::ccdb::BasicCCDBManager::instance().getCreatedNotAfter());
443447
auto local_hist = ccdb_inst.getForTimeStamp<TH1F>(ccdb_info.fullPath, options.timestamp);
444448
if (local_hist) {
445449
// case in which CCDB object contains directly a ROOT histogram

0 commit comments

Comments
 (0)