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

feat: draw PID limit boxes on eta vs. P plot in CI #273

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

c-dilks
Copy link
Member

@c-dilks c-dilks commented May 1, 2023

Briefly, what does this PR introduce?

Draw PID (eta,p) limit boxes on eta vs. P histogram (CI artifact)

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue #__)
  • Documentation update
  • Other: __

Please check if this PR fulfills the following:

  • Tests for the changes have been added
  • Documentation has been added / updated
  • Changes have been communicated to collaborators

Does this PR introduce breaking changes? What changes might users need to make to their code?

no

Does this PR change default behavior?

no

@c-dilks
Copy link
Member Author

c-dilks commented May 1, 2023

@chchatte92 could you update the p and eta ranges for the dRICH? So far I just put in some rough guesses:

limits.push_back({
"#splitline{dRICH}{gas}",
new TBox(
9.5, // p min
1.3, // eta min
50.0, // p max
3.7 // eta max
),
kRed
});
limits.push_back({
"#splitline{dRICH}{aerogel}",
new TBox(
2.0, // p min
1.3, // eta min
10.3, // p max
3.7 // eta max
),
kBlue
});

And here's what it looks like (with very low statistics):
canv_etaVsP___histos__finalState_0__ptLab_0__w_0__xF_0__y_0__z_0

@c-dilks c-dilks marked this pull request as draft May 1, 2023 17:29
@c-dilks
Copy link
Member Author

c-dilks commented May 1, 2023

Perhaps with the help of @preghenella, et al., we could add the other PID subsystems too.

@chchatte92
Copy link
Member

@c-dilks, we will do that. BTW, with Q2>1; p_T>0.1 and 0.01<y<0.95 positive pions are like this:
image
This means the complete separation (both species positively identified) of pi/K at eta=1.5 will mostly be coming from aerogel and gas will be just for vetoing. But, I remember around eta=1.5, we will have no more the aerogel ring. Won't we?

@c-dilks
Copy link
Member Author

c-dilks commented May 2, 2023

But, I remember around eta=1.5, we will have no more the aerogel ring. Won't we?

DRICH_zmin                     =      195.000
DRICH_snout_length             =       20.000
DRICH_rmax1                    =      104.656

Aerogel minimum eta:
2023-05-02-113603_657x39_scrot

@c-dilks
Copy link
Member Author

c-dilks commented May 2, 2023

BTW, you can find the updated (eta,p) plots in artifact _FULL_RESULTS under eta_vs_p_forPID.Ele.

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.

2 participants