forked from AliceO2Group/O2Physics
-
Notifications
You must be signed in to change notification settings - Fork 0
[MegaLinter] Apply linters automatic fixes to #4541 #2
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
Open
alibuild
wants to merge
202
commits into
thakurd3:mybranch
Choose a base branch
from
alibuild:alibot-cleanup-ml-4541
base: mybranch
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 hidden or 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
* Add tpcNCls and eta information for QA check * Add tpcNcls and eta information for QA check * clang-format * Rename some variables to make more descriptive * clang-format
* PWGLF: extra dev of derived V0 analysis * Add task * Please consider the following formatting changes (AliceO2Group#218) --------- Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
* Update Femto.md * MegaLinter fixes --------- Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
* Feat: Create direcotry structure in FemtoDream * Feat: fix includes * Feat: fix include headers * Fix: Override usage of multiplicity percentile
* Update tableMaker.cxx * Update tableMaker.cxx
Added PID hypothesis, global vs ITS-TPC momentum comparison & ITS chi2 vs ITS nCls histograms.
… and fix M02/NCell cut (AliceO2Group#4331) - Add a vertexQA task to investigate the quality of vertices depending on contributing tracks - Add a task to reconstruct pi0s in different calorimeter regions to extract a energy scale parameter based on their mass - Fix M02 cut, so that it is only applied for clusters with more than one cell
AliceO2Group#4329) * final touch on validation via nested loops + further port from AliPhysics * fix for wrong variable
…p#4249) * Updated task for ZDC energies * Update including fixes for coding conventions * following Anton's suggestion
…iNormalised (AliceO2Group#4336) * Increasing pT Bins for Dstar with labels & changing inequality condition for d0SoftPiNormalised * Restoring d0SoftPiNormalised inequality condition
* Introduce forward track propagation to UD * Formatting
* Modification of mixing and ITS only study * clang * Fix histogram binning * Fixes for Efficiency calculation * Fix Efficiency calculation * Efficiency correction
* check track timing for matching * Please consider the following formatting changes --------- Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
* PWGLF: add ext table spawners * Spawn only basic exts * Adjustment to configs of derived ana * Fixes to derived analysis * Please consider the following formatting changes (AliceO2Group#219) --------- Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
…selector (AliceO2Group#4319) * Add ML inference class for Lc->PKPi and related modifications to the selector * clang format * MegaLinter fixes * Implement corrections * correct clang-format --------- Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…er (AliceO2Group#4209) * Adding process full with covariance and centrality * Clang format
…2Group#4349) - Add inv mass and eta-phi histograms column by column and row by row - Minor bug fixes
* PWGCF: Update MeanPtFlucIdentified.cxx * Update CMakeLists.txt
* Utility to get CTP rates Co-authored-by: Roman Lietava <roman.lietava@cern.ch> * Add proper handling of ZDC class Co-authored-by: Roman Lietava <roman.lietava@cern.ch> * Use correct input type for ZDC class Co-authored-by: Roman Lietava <roman.lietava@cern.ch> --------- Co-authored-by: Roman Lietava <roman.lietava@cern.ch>
* PWGHF: properly define filters and partitions to speedup HF combinatorial * Please consider the following formatting changes * Apply the same for V0s and cascades and remove cut on max pT for soft pi * Factorise loop for 3 prongs and for D* * Please consider the following formatting changes --------- Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
* Add task for studying fake ITS-TPC matchings * Fix megalinter
…4522) * Adding centrality selection cuts * Adding centrality selection cuts * Adding centrality selection cuts * Adding centrality selection cuts * Adding centrality selection cuts * 3D CFs in mult and kT bins * 3D CFs in mult and kT bins * 3D CFs in mult and kT bins * Adding centrality selection cuts * Adding centrality selection cuts
* check secondary efficiency + add eta axis * Please consider the following formatting changes * add binomially test with tag-and-probe --------- Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
* Add Lc ML scores in Sc output. * Fix comment. * Fixe template name and fill BDT score only if non-empty. --------- Co-authored-by: Mattia Faggin <mfaggin@cern.ch>
…liceO2Group#4528) * Implement FIT triggers * Add new QA histograms * clang-format
* Fix missing nabs, add mixed processing for 2-prongs * Make 2-prongs the trigger particles * Limit efficiency cache allocations * Trigger histogram names
* Feat: set cut on ITS cluster separately for different particles * Feat: add separate cut on ITSncls for protons and deuterons * Fix: fix description
AliceO2Group#4527) * fixing index referencing and adding minor improvements and adjustments * fixing formating
…O2Group#4506) * PWGHF: add sparse to taskD0 to perform cut variation method * Please consider the following formatting changes * fix MegaLinter * Address Mattia's comments * Add reflections to sparse * Please consider the following formatting changes --------- Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…etqa.cxx (AliceO2Group#4535) * fix merge conflict due to old PR * adding changes in comments * remove comments * fix range * MegaLinter fixes --------- Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
…2Group#4538) * Merge data and MC analysis process * Code formatting * Merge 3-body decayed hypertirton matter and antimatter selections * Add MC production check for hypertirton in analysis process * Fix elseif position
* Adding CCDB connection * fix * fix * fix * fix * Please consider the following formatting changes --------- Co-authored-by: junleekim <junleekim@junleekims-MacBook-Pro.local> Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
* Adding more pT bins for Dstar candidates and rebinning Decaylength and Impactparameter Hists * correction to binning option * Correcting default value of impact parameter cut of soft pi * Please consider the following formatting changes * correcting spelling mistake * fix spelling * MegaLinter fixes --------- Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
3262c09 to
8049c15
Compare
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#4541 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/.