Skip to content
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

PodioSource: Adding remove, merge and createPseudoJets analyzers #427

Merged
merged 7 commits into from
Feb 26, 2025

Conversation

kjvbrt
Copy link
Contributor

@kjvbrt kjvbrt commented Feb 20, 2025

  • Adds ability to remove or add together collections of reconstructed particles.
  • Adds analyzer , which converts collection of reconstructed particles into FastJet Pseudojets.

These analyzers allow to replace this set of steps common in Jet Clustering for DataSource based analysis:

# do jet clustering on all particles, except the muons
    df = df.Define("rps_no_muons", "FCCAnalyses::ReconstructedParticle::remove(ReconstructedParticles, muons)")
    df = df.Define("RP_px", "FCCAnalyses::ReconstructedParticle::get_px(rps_no_muons)")
    df = df.Define("RP_py", "FCCAnalyses::ReconstructedParticle::get_py(rps_no_muons)")
    df = df.Define("RP_pz","FCCAnalyses::ReconstructedParticle::get_pz(rps_no_muons)")
    df = df.Define("RP_e", "FCCAnalyses::ReconstructedParticle::get_e(rps_no_muons)")
    df = df.Define("pseudo_jets", "FCCAnalyses::JetClusteringUtils::set_pseudoJets(RP_px, RP_py, RP_pz, RP_e)")

to

# do jet clustering on all particles, except the muons
    df = df.Define("rps_no_muons", "ReconstructedParticle::remove(ReconstructedParticles, muons)")
    df = df.Define("pseudo_jets", "JetClustering::createPseudoJets(rps_no_muons)")

@kjvbrt
Copy link
Contributor Author

kjvbrt commented Feb 26, 2025

Failing of ubuntu22 nightlies test expected (test files are compatible with the stable release)

@kjvbrt kjvbrt merged commit fe298f7 into HEP-FCC:master Feb 26, 2025
4 of 5 checks passed
@kjvbrt kjvbrt deleted the pseudo-jets branch February 26, 2025 14:42
Sign up for free to 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.

1 participant