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

Whether a rotation is needed in BreitFrameJet Reconstruction function #176

Open
xinbaili opened this issue Sep 7, 2022 · 1 comment
Open
Labels
help wanted Extra attention is needed

Comments

@xinbaili
Copy link

xinbaili commented Sep 7, 2022

Hi @c-dilks , in line758 of Kinematics.cxx :https://github.com/eic/sidis-eic/blob/7c134e6c628f4140e27b21901bfe05d22609f132/src/Kinematics.cxx#L758, I look into the function GetBreitFrameJets, and find there is only a boost to let the virtual photon energy equal to zero, without a rotation to make it along z axis. Would you like to consider that or elaborate more? I think there should be a rotation in that function on the kinematic as input of jet reconstruction.

@c-dilks
Copy link
Member

c-dilks commented Sep 7, 2022

We haven't tested this part in a while, so I would not be surprised if it's not correct. This GetBreitFrameJets method also requires centauro, which is not included by default. In any case, we need help with supporting jets in sidis-eic in general, since so far we have only focused on SIDIS.

@c-dilks c-dilks added the help wanted Extra attention is needed label Sep 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants