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

Large resolutions from AnalysisEpic and AnalysisEcce hadronic recon. methods #187

Open
c-dilks opened this issue Oct 4, 2022 · 5 comments
Assignees
Labels
bug Something isn't working fullsim

Comments

@c-dilks
Copy link
Member

c-dilks commented Oct 4, 2022

Resolutions from AnalysisEcce from hadronic reconstruction methods are often large, and not centered about zero. For example, y in bins of (x,Q2), compared with AnalysisDelphes and AnalysisAthena, reconstructed by the JB method (this is a CI artifact):

canv_x_q2_cov_finalState_0_ptLab_0_w_0_xF_0_y_0_z_0_y_Res JB

In contrast, the electron method seems reasonable:
canv_x_q2_cov_finalState_0_ptLab_0_w_0_xF_0_y_0_z_0_y_Res Ele

@c-dilks c-dilks added bug Something isn't working fullsim labels Oct 4, 2022
@rseidl-rcf
Copy link
Contributor

The reason is that the clusters are not added in AnalysisEcce, yet. I will try to add them in the next days.

@rseidl-rcf rseidl-rcf self-assigned this Oct 6, 2022
@c-dilks
Copy link
Member Author

c-dilks commented Oct 8, 2022

FYI #184 fixes a bug in the Q2 weighting, and resolves some of the strange features we discussed at the previous meeting, for example, the hole in the y distribution.

@c-dilks
Copy link
Member Author

c-dilks commented Oct 8, 2022

See plots from the corresponding workflow run https://github.com/eic/sidis-eic/actions/runs/3208357494
Scroll down a bit and click on _FULL_RESULTS, a zip file of plots

@c-dilks c-dilks changed the title Large resolutions from AnalysisEcce hadronic recon. methods Large resolutions from AnalysisEpic and AnalysisEcce hadronic recon. methods Nov 29, 2022
@c-dilks
Copy link
Member Author

c-dilks commented Nov 29, 2022

@Gregtom3 AnaysisEpic seems to have this issue too.

@rseidl-rcf
Copy link
Contributor

I finally got around looking at the clusters in AnalysisEcce...only to find that the clusters were not written out in the Evaluator output for that simulation. In my local Evaluator output they are included, but that is not accessible from S3. I will try to get it set up using my local output files, but in order to use the other ones we would need the simulation team to replace them once more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fullsim
Projects
None yet
Development

No branches or pull requests

2 participants