Skip to content

[PWGUD] Removed 2 process functions and fixed o2-linter issues #11243

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
rolavick merged 3 commits into from
May 19, 2025
Merged

[PWGUD] Removed 2 process functions and fixed o2-linter issues #11243

rolavick merged 3 commits into from
May 19, 2025

Conversation

@ananthapadmanabhan18
Copy link
Contributor

@ananthapadmanabhan18 ananthapadmanabhan18 commented May 19, 2025

Here are few changes that i made:

  • Removed process functions (data and MC reco) which used only the filtered tables, instead i used filtering on the main functions
  • Changed the TLorentzVector to recommended template.
  • Added few more configurables for vertex cuts and number of PV Contributors.
  • Renamed few variables
  • Only single truegapside=2 events selected

@github-actions github-actions bot added the pwgud label May 19, 2025
@github-actions
Copy link

github-actions bot commented May 19, 2025 •
edited
Loading

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

@github-actions github-actions bot changed the title Removed 2 process functions and fixed o2-linter issues [PWGUD] Removed 2 process functions and fixed o2-linter issues May 19, 2025
rolavick
rolavick requested changes May 19, 2025
View reviewed changes
vkucera
vkucera reviewed May 19, 2025
View reviewed changes
@rolavick rolavick merged commit 013c59b into AliceO2Group:master May 19, 2025
13 checks passed
ariedel-cern pushed a commit to ariedel-cern/O2Physics that referenced this pull request May 23, 2025
@ananthapadmanabhan18 @ariedel-cern
[PWGUD] Removed 2 process functions and fixed o2-linter issues (Alice…
84d27c5 
…O2Group#11243)
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Jun 11, 2025
@ananthapadmanabhan18 @jinhyunni
[PWGUD] Removed 2 process functions and fixed o2-linter issues (Alice…
7915b96 
…O2Group#11243)
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jun 12, 2025
@ananthapadmanabhan18
[PWGUD] Removed 2 process functions and fixed o2-linter issues (Alice…
f27605b 
…O2Group#11243)
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jun 14, 2025
@ananthapadmanabhan18
[PWGUD] Removed 2 process functions and fixed o2-linter issues (Alice…
b63893b 
…O2Group#11243)
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Jun 17, 2025
@ananthapadmanabhan18 @smaff92
[PWGUD] Removed 2 process functions and fixed o2-linter issues (Alice…
21ab4ac 
…O2Group#11243)
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
@ananthapadmanabhan18 @CesarOmarRA
[PWGUD] Removed 2 process functions and fixed o2-linter issues (Alice…
2c81ced 
…O2Group#11243)
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
@ananthapadmanabhan18 @jloemker
[PWGUD] Removed 2 process functions and fixed o2-linter issues (Alice…
f1316a2 
…O2Group#11243)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vkucera vkucera vkucera left review comments

@rolavick rolavick rolavick approved these changes

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

@pbuehler pbuehler Awaiting requested review from pbuehler

@abylinkin abylinkin Awaiting requested review from abylinkin

Assignees

No one assigned

Labels

pwgud

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ananthapadmanabhan18 @rolavick @vkucera