Skip to content

Conversation

@choich08365
Copy link
Contributor

bjetTreeCreator.cxx

  • Removed jet event weight task dependencies to avoid memory consumption issues

@github-actions github-actions bot added the pwgje label Nov 5, 2025
@github-actions
Copy link

github-actions bot commented Nov 5, 2025

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

@github-actions github-actions bot changed the title (b-jet tree creator) Removed jet event weight task dependencies [PWGJE] (b-jet tree creator) Removed jet event weight task dependencies Nov 5, 2025
@choich08365
Copy link
Contributor Author

Hi, @hyungjun-lee and @hahassan7,
I made a PR updating the b-jet tree creator code to use collision.weight() instead of jet.eventWeight() to avoid the current memory consumption issues on the hyperloop. I'd like you to confirm if this update works well for you. Thank you in advance.

@choich08365 choich08365 marked this pull request as ready for review November 5, 2025 13:56
@hyungjun-lee
Copy link
Contributor

Thanks, Changhwan.
Could you please ask Nima for approval? There are some “UpperCamelCase” failures at the moment, but as far as I understand, they can be safely ignored.

@hahassan7
Copy link
Collaborator

Thanks Changhwan, thats really a very good fix which avoids the use the memory hungry MCP and MCD event weight wagon.

@nzardosh nzardosh merged commit 405f868 into AliceO2Group:master Nov 7, 2025
12 of 13 checks passed
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
…es (AliceO2Group#13697)

Co-authored-by: Changhwan Choi <changhwan.choi@cern.ch>
@choich08365 choich08365 deleted the gnnbjet branch December 23, 2025 09:21
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