Skip to content

Adding Jets to the HF framework - #386

Closed
nzardosh wants to merge 4 commits into
AliceO2Group:masterfrom
nzardosh:hfjets
Closed

Adding Jets to the HF framework#386
nzardosh wants to merge 4 commits into
AliceO2Group:masterfrom
nzardosh:hfjets

Conversation

@nzardosh

Copy link
Copy Markdown
Contributor

No description provided.

Comment threadcodeHF/Compare.C Outdated
Comment on lines +212 to +214
AddHistogram(vecHisJets, "#phi", "fHistJetPhi", "jet-finder-hf-data/h_jet_phi", 1, 1, 0);
AddHistogram(vecHisJets, "#eta", "fHistJetEta", "jet-finder-hf-data/h_jet_eta", 1, 1, 0);
AddHistogram(vecHisJets, "N_{tracks}", "fHistJetNConstituents", "jet-finder-hf-data/h_jet_ntracks", 1, 1, 0);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing italics

Comment threadcodeHF/Compare.C Outdated
Comment threadcodeHF/Compare.C Outdated
Comment on lines +225 to +230
AddHistogram(vecHisJetsMC, "#phi_{det}", "fHistJetPhi", "jet-finder-hf-mcd/h_jet_phi", 1, 1, 0);
AddHistogram(vecHisJetsMC, "#phi_{gen}", "fHistJetPhi_Part", "jet-finder-hf-mcp/h_jet_phi", 1, 1, 0);
AddHistogram(vecHisJetsMC, "#eta_{det}", "fHistJetEta", "jet-finder-hf-mcd/h_jet_eta", 1, 1, 0);
AddHistogram(vecHisJetsMC, "#eta_{gen}", "fHistJetEta_Part", "jet-finder-hf-mcp/h_jet_eta", 1, 1, 0);
AddHistogram(vecHisJetsMC, "N_{tracks,det}", "fHistJetNConstituents", "jet-finder-hf-mcd/h_jet_ntracks", 1, 1, 0);
AddHistogram(vecHisJetsMC, "N_{tracks,gen}", "fHistJetNConstituents_Part", "jet-finder-hf-mcp/h_jet_ntracks", 1, 1, 0);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing italics

Comment threadcodeHF/Compare.C Outdated
Comment threadcodeHF/RunHFTaskLocal.C Outdated
Comment threadcodeHF/dpl-config_run3.json Outdated
Comment on lines +1141 to +1142
"processMCP": "true"
},

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no configuration for jet-matching to be added?

Comment threadcodeHF/workflows.yml Outdated
Comment threadcodeHF/workflows.yml
Comment threadexec/make_command_o2.py Outdated
Comment on lines +267 to +268
msg_err("Workflow %s is already present." % string_wf)
# sys.exit(1)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
msg_err("Workflow %s is already present."%string_wf)
# sys.exit(1)
msg_warn("Workflow %s is already present."%string_wf)

qgpand others added 3 commits February 14, 2023 11:04
adding hf-jets
pre-rebase commit
adding HF-jet comparisons
intermdeiate
Adding Jets to the HF framework
Comment on lines +7 to +9
INPUT_CASE=3 # Input case

NFILESMAX=1 # Maximum number of processed input files. (Set to -0 to process all; to -N to process all but the last N files.)
NFILESMAX=50 # Maximum number of processed input files. (Set to -0 to process all; to -N to process all but the last N files.)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not commit changes of the default settings

DOO2_SKIM=1 # hf-track-index-skim-creator
DOO2_CAND_2PRONG=1 # hf-candidate-creator-2prong
DOO2_CAND_3PRONG=1 # hf-candidate-creator-3prong
DOO2_CAND_3PRONG=0 # hf-candidate-creator-3prong

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not commit changes of the default settings

@vkuceravkucera mentioned this pull request Sep 9, 2023
@vkuceravkucera closed this Sep 9, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@nzardosh@vkucera@qgp