Skip to content

Treatment of electron jets #121

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
samvanstroud opened this issue Sep 15, 2022 · 3 comments
Open

Treatment of electron jets #121

samvanstroud opened this issue Sep 15, 2022 · 3 comments

Comments

@samvanstroud
Copy link
Contributor

In most samples we write n_truth_promptLepton which can discriminate between mislabelled electron-jets and light-jets. In the GN1 plotting code (which we will stop using in favour of this package) we require all jets to have n_truth_promptLepton == 0.

We might consider (optionally) doing the the same here to avoid inflating light rejection at high pT (when electron jets become a significant fraction of light jets).

More info here: https://its.cern.ch/jira/browse/AFT-593

@joschkabirk
Copy link
Contributor

Hi @samvanstroud,

great to see that you will make use of this package!

I think, however, that this should not be included in puma itself, but rather in the GNN-specific plotting scripts that are used in the end?

But let's see what @afroch, @manuguth and @philippgadow think.

@samvanstroud
Copy link
Contributor Author

I guess we are really having the same discussion as in #122. But I would note that removing the electron jets should be done when evaluating performance for all taggers, and not just GN1. Since there are several "suggested ways of doing things" in terms of these selections and e.g. discriminant calculations, it makes sense to me to have this package make these things as easy as possible for the user. But let's continue the discussion in #122 :)

@samvanstroud
Copy link
Contributor Author

Some progress: #165 added this selection in the high level example docs https://umami-hep.github.io/puma/main/examples/high_level_api.html#initialising-the-taggers

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

No branches or pull requests

2 participants