Skip to content

digitizer: Ability to fetch bcFilling scheme from CCDB#8378

Merged
sawenzel merged 1 commit intoAliceO2Group:devfrom
sawenzel:swenzel/bcfillingccdb
Mar 18, 2022
Merged

digitizer: Ability to fetch bcFilling scheme from CCDB#8378
sawenzel merged 1 commit intoAliceO2Group:devfrom
sawenzel:swenzel/bcfillingccdb

Conversation

@sawenzel
Copy link
Copy Markdown
Collaborator

No description provided.

@sawenzel sawenzel requested a review from shahor02 as a code owner March 17, 2022 10:25
if (bcPatternString == "ccdb") {
LOG(info) << "Fetch bcPattern information from CCDB";
// fetch the GRP Object
auto& ccdb = o2::ccdb::BasicCCDBManager::instance();
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aren't you planning to move this class to DPL CCDB backend? Then this code cannot be in the init() but should move to run(). In any case, I think one should at least set the ccdb host (the GRPLHCIFs are in the http://alice-ccdb.cern.ch).
Also, with this code you get the object for now(), for the anchoring one should pass the run timestamp.
Since you have already headers creation / firstTFOrbit fixed, this would work automatically with the DPL backend.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything is set up correctly for CCDBManager here (in the outside workflow creation) and I will use CCDBManager until the caching from snapshots is working correctly (which I think is not yet the case).

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, I did not find where it is set. What exactly does not work with caching? I am already working with export ALICEO2_CCDB_LOCALCACHE=<xxx> to create the cache and with --condition-backend file://<xxx> to use it directly (also can be done via export DPL_CONDITION_BACKEND=file://<xxx>)

@sawenzel sawenzel merged commit cce7638 into AliceO2Group:dev Mar 18, 2022
@sawenzel sawenzel deleted the swenzel/bcfillingccdb branch March 18, 2022 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants