Propagate MC switch in the AliPhysics task#264
Merged
vkucera merged 1 commit intoAliceO2Group:masterfrom Jun 21, 2021
Merged
Conversation
vkucera
added a commit
that referenced
this pull request
Jun 23, 2021
* Adjust plot sizes (#233) Co-authored-by: Benedikt Volkel <benedikt.volkel@cern.ch> * Update README (#234) Co-authored-by: Benedikt Volkel <benedikt.volkel@cern.ch> * Fix QA simple workflow (#236) * Add new features in make_command_o2 (#242) * Support real-data-only options/tables and workflow aliases * Print one workflow per line in script_o2 * Add tables of selectors * Detect duplicate workflows * Add Λc → K0S p (#243) * Add Lc -> K0S p workflows. * Add LcK0sp tables * Add Λc → K0S p, part 2 (#244) * Fix typo (#246) * Add code owners (#252) * Add event-selection task in runtest (#192) * Add event-selection tasks and adapt json files * Add entry for histo of event selection * fix conflicts * Update config_tasks.sh * Fix defaults * Fix comments * Update onia json * Use camel case * Update name of directory in O2 AnalysisResults * Remove unused entry * Fix conflict * Revert removal of entry * Make explicit the activation of the event selection * Trim trailing whitespace * fix workflow names * fix names * Add event seletion to workflows * Fix directory name * Remove wrong file * Modify list of dependencies in case of hf-track-index-skims-creator * Add possibility to run together 2/3-prongs and cascades * Implement Vit's suggestions * Implement Vit's suggestions (part 2) * Remove unneeded workflows Co-authored-by: Vít Kučera <vit.kucera@cern.ch> * Fix workflow database * Polish config_tasks.sh * Fix O2 command format and workflow duplication detection * Make "activate" key in workflow database optional * Make O2 exit with 1 for any error * Run postprocessing in ROOT environment * Fix postprocessing environment detection * Plot prompt/non-prompt efficiency * Add efficiency validation plots (#260) * Add efficiency validation plots * Adjust labels * Small updates for D+ analysis (#258) * Small update of Dplus comparison files * Update config file * Remove trailing whitespaces * Propagate changes to other .json files Co-authored-by: Vít Kučera <vit.kucera@cern.ch> * Add efficiency validation options * Fix analysis README * Fix Python * Remove table of content from README (generated automatically) * Update linter instructions * Add instructions for analysis code contributors * Update documentation * Simplify batch scripts * Catch more O2 errors * Update reference plots * Update clean.sh * Propagate MC switch in the AliPhysics task (#264) * Rename Style class (#266) Co-authored-by: Benedikt Volkel <benedikt.volkel@cern.ch> * Fix workflow configuration * Catch more O2 errors (#267) Co-authored-by: benedikt-voelkel <benedikt.voelkel@gmx.de> Co-authored-by: Benedikt Volkel <benedikt.volkel@cern.ch> Co-authored-by: Fabrizio <fabrizio.grosa@cern.ch> Co-authored-by: Fabio Catalano <fabio.catalano@cern.ch>
vkucera
added a commit
that referenced
this pull request
Aug 16, 2021
* add X cand creator * Add first version of X selector add TOF pid for X selector to json files add pT dependent selection cuts to json files * propagate X options to all json files * remove unused variables from json for LcpKpi (#227) Co-authored-by: Luigi <ldellost@aliceml.cern.ch> * Fix formatting * Fix empty spaces * add X(3872) to yaml database (#231) path to results still needed also fixed a typo in the Xi_cc histo list * Fix workflow for X * Update onia file * lower pt min of electrons * Commit new cuts and improved plotting * Fix pylint * new datasets * [Mega-Linter] Apply linters fixes * Add input slot for MuonID sample * Add ALICE 2/3 switch in Jpsi selector (#239) * Fix path to Onia MID production * Add ALICE 2/3 switch in Jpsi selector * Remove ee from Jpsi selector strings * Update path onia * Add settings for muon analysis (#245) Co-authored-by: Gian Michele Innocenti <ginnocen@alicecerno2.cern.ch> * Fix eta and update input files (#247) Co-authored-by: Gian Michele Innocenti <ginnocen@alicecerno2.cern.ch> * Add plotting macro for contamination (#248) Co-authored-by: Gian Michele Innocenti <ginnocen@alicecerno2.cern.ch> * Add macro for contamination study (#249) * [Mega-Linter] Apply linters fixes * Run5 dev (#250) * Add qa-rejection (#251) * Add is_e_not_pi_plots (#253) * Use a single variable for input file path (#254) * Use full production for case 8 (#255) * use working input * Sync with master (#268) * Adjust plot sizes (#233) Co-authored-by: Benedikt Volkel <benedikt.volkel@cern.ch> * Update README (#234) Co-authored-by: Benedikt Volkel <benedikt.volkel@cern.ch> * Fix QA simple workflow (#236) * Add new features in make_command_o2 (#242) * Support real-data-only options/tables and workflow aliases * Print one workflow per line in script_o2 * Add tables of selectors * Detect duplicate workflows * Add Λc → K0S p (#243) * Add Lc -> K0S p workflows. * Add LcK0sp tables * Add Λc → K0S p, part 2 (#244) * Fix typo (#246) * Add code owners (#252) * Add event-selection task in runtest (#192) * Add event-selection tasks and adapt json files * Add entry for histo of event selection * fix conflicts * Update config_tasks.sh * Fix defaults * Fix comments * Update onia json * Use camel case * Update name of directory in O2 AnalysisResults * Remove unused entry * Fix conflict * Revert removal of entry * Make explicit the activation of the event selection * Trim trailing whitespace * fix workflow names * fix names * Add event seletion to workflows * Fix directory name * Remove wrong file * Modify list of dependencies in case of hf-track-index-skims-creator * Add possibility to run together 2/3-prongs and cascades * Implement Vit's suggestions * Implement Vit's suggestions (part 2) * Remove unneeded workflows Co-authored-by: Vít Kučera <vit.kucera@cern.ch> * Fix workflow database * Polish config_tasks.sh * Fix O2 command format and workflow duplication detection * Make "activate" key in workflow database optional * Make O2 exit with 1 for any error * Run postprocessing in ROOT environment * Fix postprocessing environment detection * Plot prompt/non-prompt efficiency * Add efficiency validation plots (#260) * Add efficiency validation plots * Adjust labels * Small updates for D+ analysis (#258) * Small update of Dplus comparison files * Update config file * Remove trailing whitespaces * Propagate changes to other .json files Co-authored-by: Vít Kučera <vit.kucera@cern.ch> * Add efficiency validation options * Fix analysis README * Fix Python * Remove table of content from README (generated automatically) * Update linter instructions * Add instructions for analysis code contributors * Update documentation * Simplify batch scripts * Catch more O2 errors * Update reference plots * Update clean.sh * Propagate MC switch in the AliPhysics task (#264) * Rename Style class (#266) Co-authored-by: Benedikt Volkel <benedikt.volkel@cern.ch> * Fix workflow configuration * Catch more O2 errors (#267) Co-authored-by: benedikt-voelkel <benedikt.voelkel@gmx.de> Co-authored-by: Benedikt Volkel <benedikt.volkel@cern.ch> Co-authored-by: Fabrizio <fabrizio.grosa@cern.ch> Co-authored-by: Fabio Catalano <fabio.catalano@cern.ch> * Add ALICE3 switch (#269) * Fixes * Add Jpsi -> mumu preselection cuts in JSON * Fix missing columns in JSON Co-authored-by: rspijkers <rik.spijkers@gmail.com> Co-authored-by: DelloStritto <47105254+DelloStritto@users.noreply.github.com> Co-authored-by: Luigi <ldellost@aliceml.cern.ch> Co-authored-by: Gian Michele Innocenti <ginnocen@alicecerno2.cern.ch> Co-authored-by: Rik Spijkers <78484875+rspijkers@users.noreply.github.com> Co-authored-by: ginnocen <ginnocen@users.noreply.github.com> Co-authored-by: Gian Michele Innocenti <gminnocen@gmail.com> Co-authored-by: benedikt-voelkel <benedikt.voelkel@gmx.de> Co-authored-by: Benedikt Volkel <benedikt.volkel@cern.ch> Co-authored-by: Fabrizio <fabrizio.grosa@cern.ch> Co-authored-by: Fabio Catalano <fabio.catalano@cern.ch>
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
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.
Related PR: alisw/AliPhysics#17964