forked from AliceO2Group/O2Physics
-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
[MegaLinter] Apply linters automatic fixes to #4661 #18
Open
alibuild
wants to merge
27
commits into
AlexBigO:dev_bmesons
Choose a base branch
from
alibuild:alibot-cleanup-ml-4661
base: dev_bmesons
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* PWGHF: Add nSigma TPC TOF to Ds tree creator output * MegaLinter fixes * PWGHF: Add D+ -> KKPi flag * Please consider the following formatting changes * Add comments and fixed formatting * Please consider the following formatting changes * Refactored candidateCreator D meson selections * MegaLinter fixes * Fix formatting * Imlemented suggestions --------- Co-authored-by: ALICE Action Bot <[email protected]>
* PWGLF: Add centrality QA/testing mode for der data * Please consider the following formatting changes (AliceO2Group#234) * Add feeddown matrix calculation for first test * Please consider the following formatting changes (AliceO2Group#235) * Add plain QA mode for cross-checks * Fix minor bugs introduced in last commit --------- Co-authored-by: ALICE Builder <[email protected]>
* first commit of task forcascade flow analysis * fix to make it run over strangeness derived data * add process MC * add selection on cascade candidates for signal training * Formatting and fix cut counter Co-authored-by: Maximiliano Puccio <[email protected]> * Please consider the following formatting changes --------- Co-authored-by: Chiara De Martin <[email protected]> Co-authored-by: Maximiliano Puccio <[email protected]> Co-authored-by: ALICE Action Bot <[email protected]>
* Update r2p2-4-id.cxx Condensed the PID functions into one common template accepting the Configurable cuts. Added Process for MC data to check PID efficiency. Reduced the r2p2 task to now run for one kind of identified pair at a time. Also added functionality to find cross-correlations between two pt ranges. * Update r2p2-4-id.cxx
* modify filter conditions * modify the code according to the reviews of PR#4562
- add missing break
* First step towards identified analysis * Fixed first wave of MegaLinter errors --------- Co-authored-by: Victor <[email protected]>
Co-authored-by: Georgios Mantzaridis <[email protected]>
Also collisions within ITS read out frame border are rejected Co-authored-by: Victor <[email protected]>
* PWGLF: extra event info for debug tests * Please consider the following formatting changes (AliceO2Group#236) --------- Co-authored-by: ALICE Builder <[email protected]>
Update: Added comparison for tpcinnerparam with global pT
Co-authored-by: Victor <[email protected]>
* bug fix * including TOF and ITS selection histos again after the messed rebase * adding forgotten TOF counter * TOF counter bug fix * small reshuffle * increasing precision of axis * MegaLinter fixes * adding function which mimics GlobatTracks flag using UD tables * MegaLinter fixes * fixing redeclaration --------- Co-authored-by: ALICE Action Bot <[email protected]>
* Change cmake * updated tasks with modification * Fixed compilation errors * modifying task * Updated task for new checks and removed unrequired parts for AQC * bug fixed CMakeList.txt * fixed format * fixed formatting and megalinter issue * Macro to test FIT signal for UPC * Updated for inclusive UPC event selection * Updated Task PWGUD/AQC * resolving commit issue * resolving formatting for Cmakelist * New Tasks to study FIT veto and ZDC for UPC --------- Co-authored-by: akhatun <[email protected]> Co-authored-by: akhatun <[email protected]>
* fix axes for sparse * titles on THnSparse axes * title definitions for TOF sparse * obligatory whitespace
* Add info globalTrakWoDca * Remove new variable from candidateCreator * Update PWGHF/TableProducer/treeCreatorToXiPi.cxx Co-authored-by: Fabio Catalano <[email protected]> * Update PWGHF/TableProducer/treeCreatorToXiPi.cxx Co-authored-by: Fabio Catalano <[email protected]> * Update PWGHF/TableProducer/treeCreatorToXiPi.cxx Co-authored-by: Fabio Catalano <[email protected]> * Update PWGHF/TableProducer/treeCreatorToXiPi.cxx Co-authored-by: Fabio Catalano <[email protected]> --------- Co-authored-by: Fabio Catalano <[email protected]>
Co-authored-by: Andrea <andrea.rossicern.ch>
* added code for kshort kshort * changed the name from KshortKshort to kshortkshort since the name of the executabel cannot contain capital letters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Your PR AliceO2Group#4661 cannot be merged as is. You should either run MegaLinter yourself and update the pull request, or merge this PR in yours.
You can find how to run MegaLinter locally at https://megalinter.io/latest/mega-linter-runner/.