Skip to content

[PWGCF] added some more efficiency histograms#16900

Merged
SwatiSaha-1997 merged 28 commits into
AliceO2Group:masterfrom
nidamalikk:master
Jul 2, 2026
Merged

[PWGCF] added some more efficiency histograms#16900
SwatiSaha-1997 merged 28 commits into
AliceO2Group:masterfrom
nidamalikk:master

Conversation

@nidamalikk

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the pwgcf label Jul 2, 2026
@github-actions github-actions Bot changed the title added some more efficiency histograms [PWGCF] added some more efficiency histograms Jul 2, 2026
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

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

@alibuild

alibuild commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for d35cf80 at 2026-07-02 12:08:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 18 errors and 70 warnings.'
Found 18 errors and 70 warnings.
++ [[ 18 -gt 0 ]]
++ cat /sw/BUILD/ad2a712c6abcf331d59a53201552bb779c87d862/O2Physics-code-check/errors.txt
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:61:8: error: constructor does not initialize these fields: pdgService, ccdb [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:65:78: error: macro argument should be enclosed in parentheses [bugprone-macro-parentheses,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:406:49: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:413:50: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:421:54: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:428:55: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:437:52: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:444:53: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:452:7: error: use auto when initializing with a template cast to avoid duplicating the type name [modernize-use-auto,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:453:23: error: do not use reinterpret_cast [cppcoreguidelines-pro-type-reinterpret-cast,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:454:23: error: do not use reinterpret_cast [cppcoreguidelines-pro-type-reinterpret-cast,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:863:16: error: narrowing conversion from 'double' to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:946:9: error: boolean expression can be simplified by DeMorgan's theorem [readability-simplify-boolean-expr,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1061:9: error: boolean expression can be simplified by DeMorgan's theorem [readability-simplify-boolean-expr,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1147:16: error: narrowing conversion from 'double' to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1271:16: error: narrowing conversion from 'float' to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1286:16: error: narrowing conversion from 'double' to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1361:16: error: narrowing conversion from 'double' to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
++ [[ 70 -gt 0 ]]
++ cat /sw/BUILD/ad2a712c6abcf331d59a53201552bb779c87d862/O2Physics-code-check/warnings.txt
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:466:68: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:468:69: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:474:75: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:476:76: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:482:70: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:484:71: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:495:45: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:520:80: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:522:74: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:524:78: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:526:78: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:528:72: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:530:92: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:567:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:569:45: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:571:42: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:573:58: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:575:27: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:577:56: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:579:53: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:581:61: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:583:52: warning: statement should be inside braces [readability-braces-around-statements]
[0 more errors; see full log]

Full log here.

@alibuild

alibuild commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 14475d4 at 2026-07-02 12:19:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 18 errors and 70 warnings.'
Found 18 errors and 70 warnings.
++ [[ 18 -gt 0 ]]
++ cat /sw/BUILD/c2766c40f54bc26ed4d0484d0f41c107a3ebf145/O2Physics-code-check/errors.txt
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:61:8: error: constructor does not initialize these fields: pdgService, ccdb [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:65:78: error: macro argument should be enclosed in parentheses [bugprone-macro-parentheses,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:406:49: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:413:50: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:421:54: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:428:55: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:437:52: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:444:53: error: 'data' should be used for accessing the data pointer instead of taking the address of the 0-th element [readability-container-data-pointer,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:452:7: error: use auto when initializing with a template cast to avoid duplicating the type name [modernize-use-auto,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:453:23: error: do not use reinterpret_cast [cppcoreguidelines-pro-type-reinterpret-cast,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:454:23: error: do not use reinterpret_cast [cppcoreguidelines-pro-type-reinterpret-cast,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:863:16: error: narrowing conversion from 'double' to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:946:9: error: boolean expression can be simplified by DeMorgan's theorem [readability-simplify-boolean-expr,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1061:9: error: boolean expression can be simplified by DeMorgan's theorem [readability-simplify-boolean-expr,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1147:16: error: narrowing conversion from 'double' to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1271:16: error: narrowing conversion from 'float' to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1286:16: error: narrowing conversion from 'double' to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1361:16: error: narrowing conversion from 'double' to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
++ [[ 70 -gt 0 ]]
++ cat /sw/BUILD/c2766c40f54bc26ed4d0484d0f41c107a3ebf145/O2Physics-code-check/warnings.txt
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:466:68: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:468:69: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:474:75: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:476:76: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:482:70: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:484:71: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:495:45: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:520:80: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:522:74: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:524:78: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:526:78: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:528:72: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:530:92: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:567:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:569:45: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:571:42: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:573:58: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:575:27: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:577:56: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:579:53: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:581:61: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:583:52: warning: statement should be inside braces [readability-braces-around-statements]
[0 more errors; see full log]

Full log here.

@alibuild

alibuild commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 10318ba at 2026-07-02 13:12:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 9 errors and 33 warnings.'
Found 9 errors and 33 warnings.
++ [[ 9 -gt 0 ]]
++ cat /sw/BUILD/a4ea4af02137683288c3ac666a572dd91b36de8b/O2Physics-code-check/errors.txt
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:66:22: error: macro argument should be enclosed in parentheses [bugprone-macro-parentheses,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:453:23: error: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-static-cast-downcast,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:454:23: error: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-static-cast-downcast,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:869:16: error: narrowing conversion from 'Double_t' (aka 'double') to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:957:9: error: boolean expression can be simplified by DeMorgan's theorem [readability-simplify-boolean-expr,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1079:9: error: boolean expression can be simplified by DeMorgan's theorem [readability-simplify-boolean-expr,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1171:16: error: narrowing conversion from 'Double_t' (aka 'double') to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1315:16: error: narrowing conversion from 'Double_t' (aka 'double') to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1392:16: error: narrowing conversion from 'Double_t' (aka 'double') to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
++ [[ 33 -gt 0 ]]
++ cat /sw/BUILD/a4ea4af02137683288c3ac666a572dd91b36de8b/O2Physics-code-check/warnings.txt
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:466:68: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:468:69: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:474:75: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:476:76: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:482:70: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:484:71: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:495:45: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:520:80: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:522:74: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:524:78: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:526:78: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:528:72: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:530:92: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:567:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:569:45: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:571:42: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:573:58: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:575:27: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:577:56: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:579:53: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:581:61: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:583:52: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:585:52: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1272:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1274:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1276:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1278:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1280:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1282:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1284:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1286:32: warning: statement should be inside braces [readability-braces-around-statements]
[0 more errors; see full log]

Full log here.

@SwatiSaha-1997

Copy link
Copy Markdown
Collaborator

Hi @nidamalikk, there are these errors in you PR, would you kindly look at them and fix.

@nidamalikk

Copy link
Copy Markdown
Contributor Author

Hi @nidamalikk, there are these errors in you PR, would you kindly look at them and fix.

Hello Swati di, I hope it is fixed now. Thank Youuu.

@alibuild

alibuild commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for bbe2239 at 2026-07-02 15:55:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 4 errors and 10 warnings.'
Found 4 errors and 10 warnings.
++ [[ 4 -gt 0 ]]
++ cat /sw/BUILD/6657ebdc22dc2188b7deb0c437405616ad184a4b/O2Physics-code-check/errors.txt
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:887:16: error: narrowing conversion from 'Double_t' (aka 'double') to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1190:16: error: narrowing conversion from 'Double_t' (aka 'double') to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1334:16: error: narrowing conversion from 'Double_t' (aka 'double') to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1411:16: error: narrowing conversion from 'Double_t' (aka 'double') to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
++ [[ 10 -gt 0 ]]
++ cat /sw/BUILD/6657ebdc22dc2188b7deb0c437405616ad184a4b/O2Physics-code-check/warnings.txt
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1291:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1293:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1295:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1297:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1299:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1301:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1303:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1305:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1307:32: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/netchargeFluctuations.cxx:1309:32: warning: statement should be inside braces [readability-braces-around-statements]
++ [[ 4 -gt 0 ]]
++ exit 1
--

Full log here.

@SwatiSaha-1997

Copy link
Copy Markdown
Collaborator

Dear @vkucera, could you please look into it, if anything can be done to bring the git host back.
Screenshot from 2026-07-02 20-23-39

@vkucera

vkucera commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

@SwatiSaha-1997 Where do you see that? A link would be better than a screenshot.

@SwatiSaha-1997

SwatiSaha-1997 commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

@SwatiSaha-1997 Where do you see that? A link would be better than a screenshot.

Dear @vkucera , please find the link https://ali-ci.cern.ch/alice-build-logs/AliceO2Group/O2Physics/16900/1567c10dbad9016d92b35ea6b85eece4ec5da5ff/build_O2Physics_o2_macOS/pretty.html
I got it by clicking on build/O2Physics/o2/macOS check that has failed

@vkucera

vkucera commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

@SwatiSaha-1997 Please ignore the macOS build. It always fails.

@SwatiSaha-1997

Copy link
Copy Markdown
Collaborator

@SwatiSaha-1997 Please ignore the macOS build. It always fails.

Thank you very much @vkucera . So, I can approve this PR as all other checks have passed.

@SwatiSaha-1997 SwatiSaha-1997 merged commit b9d05e0 into AliceO2Group:master Jul 2, 2026
14 of 15 checks passed
@vkucera

vkucera commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

@SwatiSaha-1997 But the title is too vague.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

4 participants