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

Iss930 Kalman code for incorporating ECAL energy information into track fits #992

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

robertprestonjohnson
Copy link
Collaborator

Issue 930
New driver ReconstructedParticleRefitter and code in the Kalman package to facilitate refitting tracks read back from the DST, including options to remove layers and to incorporate the ECAL energy information into the Kalman track fit.

robertprestonjohnson and others added 22 commits November 17, 2022 16:54
A Driver used to refit ReconstructedParticles by including the energy of the associated ECal cluster in the track fit. A new ReconstructedParticle collection is added to the event.
…o iss930

Incorporate a new driver template added by Norman for adding ECAL constraint to SVT tracks.
@robertprestonjohnson robertprestonjohnson self-assigned this May 23, 2023
@robertprestonjohnson robertprestonjohnson linked an issue May 23, 2023 that may be closed by this pull request
@robertprestonjohnson
Copy link
Collaborator Author

See presentation at recent HPS collaboration meeting: https://indico.slac.stanford.edu/event/8012/contributions/6023/

Copy link
Contributor

@normangraf normangraf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adds new functionality, shouldn't affect existing reconstruction.
Please proceed.

_materialManager = new MaterialSupervisor();
_materialManager.buildModel(det);

fm = det.getFieldMap();
/*
System.out.format("B field map vs y:\n");

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It appears to me lines 204 through 226 are for printing the b-field for debugging purposes. It was commented out before, can we do that again, or add a debug parameter to make it configurable to dump this info and make the default to not print this all out to stdout?

@bloodyyugo bloodyyugo self-assigned this Sep 16, 2024
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.

Add energy constraint to Kalman tracks
4 participants