Skip to content

[Common] Adjust CCDB loading to capture Run 2 GRPObject for MagField #13384

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
ddobrigk merged 6 commits into from
Oct 15, 2025
Merged

[Common] Adjust CCDB loading to capture Run 2 GRPObject for MagField #13384

ddobrigk merged 6 commits into from
Oct 15, 2025

Conversation

@ddobrigk
Copy link
Collaborator

@ddobrigk ddobrigk commented Oct 14, 2025

@alicja-pp thanks for reporting! :-)

@github-actions github-actions bot changed the title Adjust CCDB loading to cature Run 2 GRPObject for MagField [Common] Adjust CCDB loading to cature Run 2 GRPObject for MagField Oct 14, 2025
@github-actions
Copy link

github-actions bot commented Oct 14, 2025 •
edited
Loading

O2 linter results: ❌ 2 errors, ⚠️ 3 warnings, 🔕 0 disabled

@alibuild alibuild mentioned this pull request Oct 14, 2025
Merged
@ddobrigk ddobrigk marked this pull request as ready for review October 14, 2025 14:53
@ddobrigk ddobrigk requested review from a team, alibuild, iarsene, jgrosseo and ktf as code owners October 14, 2025 14:53
@ddobrigk ddobrigk changed the title [Common] Adjust CCDB loading to cature Run 2 GRPObject for MagField [Common] Adjust CCDB loading to capture Run 2 GRPObject for MagField Oct 14, 2025
@ddobrigk ddobrigk enabled auto-merge (squash) October 14, 2025 14:54
alibuild
alibuild previously approved these changes Oct 14, 2025
View reviewed changes
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

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

Auto-approving on behalf of @ddobrigk.

@ddobrigk ddobrigk disabled auto-merge October 14, 2025 15:10
@ddobrigk ddobrigk enabled auto-merge (squash) October 14, 2025 15:11
alibuild
alibuild previously approved these changes Oct 14, 2025
View reviewed changes
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

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

Auto-approving on behalf of @ddobrigk.

@alibuild
Copy link
Collaborator

alibuild commented Oct 14, 2025

Error while checking build/O2Physics/o2 for 2ca4a18 at 2025-10-14 17:23:

## sw/BUILD/O2Physics-latest/log
/sw/slc9_x86-64/GCC-Toolchain/v14.2.0-alice2-1/include/c++/14.2.0/type_traits:1493:30: error: invalid use of incomplete type 'class o2::parameters::GRPObject'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/BasicCCDBManager.h:245:48: error: no matching function for call to 'o2::ccdb::CcdbApi::retrieveFromTFileAny<o2::parameters::GRPObject>(const std::string&, o2::ccdb::CCDBManagerInstance::MD&, long int&, o2::ccdb::CCDBManagerInstance::MD*, const char [1], std::string, std::string)'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/CcdbApi.h:183:77: error: 'value' is not a member of 'std::is_base_of<o2::conf::ConfigurableParam, o2::parameters::GRPObject>'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/BasicCCDBManager.h:267:48: error: no matching function for call to 'o2::ccdb::CcdbApi::retrieveFromTFileAny<o2::parameters::GRPObject>(const std::string&, o2::ccdb::CCDBManagerInstance::MD&, long int&, o2::ccdb::CCDBManagerInstance::MD*, std::string&, std::string, std::string)'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/CcdbApi.h:183:77: error: 'value' is not a member of 'std::is_base_of<o2::conf::ConfigurableParam, o2::parameters::GRPObject>'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/CcdbApi.h:666:3: error: 'value' is not a member of 'std::is_base_of<o2::conf::ConfigurableParam, o2::parameters::GRPObject>'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/BasicCCDBManager.h:273:109: error: 'value' is not a member of 'std::is_base_of<o2::conf::ConfigurableParam, o2::parameters::GRPObject>'
/sw/slc9_x86-64/GCC-Toolchain/v14.2.0-alice2-1/include/c++/14.2.0/type_traits:1493:30: error: invalid use of incomplete type 'class o2::parameters::GRPObject'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/BasicCCDBManager.h:245:48: error: no matching function for call to 'o2::ccdb::CcdbApi::retrieveFromTFileAny<o2::parameters::GRPObject>(const std::string&, o2::ccdb::CCDBManagerInstance::MD&, long int&, o2::ccdb::CCDBManagerInstance::MD*, const char [1], std::string, std::string)'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/CcdbApi.h:183:77: error: 'value' is not a member of 'std::is_base_of<o2::conf::ConfigurableParam, o2::parameters::GRPObject>'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/BasicCCDBManager.h:267:48: error: no matching function for call to 'o2::ccdb::CcdbApi::retrieveFromTFileAny<o2::parameters::GRPObject>(const std::string&, o2::ccdb::CCDBManagerInstance::MD&, long int&, o2::ccdb::CCDBManagerInstance::MD*, std::string&, std::string, std::string)'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/CcdbApi.h:183:77: error: 'value' is not a member of 'std::is_base_of<o2::conf::ConfigurableParam, o2::parameters::GRPObject>'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/CcdbApi.h:666:3: error: 'value' is not a member of 'std::is_base_of<o2::conf::ConfigurableParam, o2::parameters::GRPObject>'
/sw/slc9_x86-64/O2/daily-20251014-0000-local1/include/CCDB/BasicCCDBManager.h:273:109: error: 'value' is not a member of 'std::is_base_of<o2::conf::ConfigurableParam, o2::parameters::GRPObject>'
ninja: build stopped: subcommand failed.

Full log here.

@alibuild alibuild mentioned this pull request Oct 14, 2025
Merged
@ddobrigk ddobrigk disabled auto-merge October 14, 2025 18:10
@ddobrigk ddobrigk enabled auto-merge (squash) October 14, 2025 18:10
alibuild
alibuild approved these changes Oct 14, 2025
View reviewed changes
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

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

Auto-approving on behalf of @ddobrigk.

@ddobrigk ddobrigk merged commit 9add62c into AliceO2Group:master Oct 15, 2025
13 of 14 checks passed
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
@ddobrigk @alibuild @ThePhDane
[Common] Adjust CCDB loading to capture Run 2 GRPObject for MagField (A…
477a91c 
…liceO2Group#13384)

Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
@ddobrigk @alibuild
[Common] Adjust CCDB loading to capture Run 2 GRPObject for MagField (A…
a388cf9 
…liceO2Group#13384)

Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alibuild alibuild alibuild approved these changes

@jgrosseo jgrosseo Awaiting requested review from jgrosseo jgrosseo is a code owner

@iarsene iarsene Awaiting requested review from iarsene iarsene is a code owner

@ktf ktf Awaiting requested review from ktf ktf is a code owner

Assignees

No one assigned

Labels

common

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ddobrigk @alibuild