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

Update PhysicsRun2021_pass0_recon.lcsim #1067

Merged
merged 1 commit into from
Dec 18, 2024
Merged

Update PhysicsRun2021_pass0_recon.lcsim #1067

merged 1 commit into from
Dec 18, 2024

Conversation

rodwyer100
Copy link
Contributor

I am adding validated kalman filter tracking algorithm parameter optimizations to the steering file for the first pass. The optimization method you are familiar with (likely from the last collaboration meeting) was used w/ 1000 iterations to obtain settings which improved MC efficiency 4% and increased data track number 10% in run 14185. I believe the reason MC only increased marginally compared to data track number is because MC already is quite clean and easy to construct well. The info pertaining to MC optimization (and its validation) was already included in previous talks. I had to manually include new optimization constraints for data, because I realized that some parameters relevant to data could not have been optimized for MC. The reward function was updated to optimized efficiency in MC and track no in a single data file in run 14185; the MC efficiency is not increasing as high as it did before after 1000 iterations but I suspect as much is expected because now it has to simultaneously work in a clean and messy environment. I am doing some further validation, but I would be comfortable using this steering file in the run. Here is a plot showing the track number increase between the two iterations:

image
Red is old, blue is new; it is shifted into higher track no per event. In total we find 10% more tracks while retaining the same MC efficiency and negligible change in fake rate or duplicates (atleast when training on 14185). I can characterize its behavior on other runs, and more extensively if requested.

Adding the optimized Kalman tracking parameters; optimized to maximize MC efficiency and data track no
@rodwyer100 rodwyer100 requested review from mgignac, sarahgaiser and bloodyyugo and removed request for sarahgaiser December 16, 2024 00:43
@bloodyyugo bloodyyugo merged commit 6214c6b into master Dec 18, 2024
1 check passed
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.

3 participants