-
Notifications
You must be signed in to change notification settings - Fork 4.7k
[Phase2 L1T] GMT Upgrade of Kalman Filter for Barrel Muon KMTF #51111
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
Open
zhenbinwu
wants to merge
46
commits into
cms-sw:master
Choose a base branch
from
zhenbinwu:Kalman_26
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
7f7c6b0
Merged AMv2.3 from repository jfernan2 with cms-merge-topic
delanocamposs a99796d
bitshift bug fix KMTF.cc
delanocamposs 584d706
New typedef for 5 dimensional covariance matrix
delanocamposs 76a32a8
new class version for KMTFTrack.h update version 4.
delanocamposs e98e4a9
initial development commits
delanocamposs e514aeb
symmetrix matrix fix
delanocamposs e5b0f3c
matrix vector fix
delanocamposs afc1d61
revert matrix vector argument to upper triangle only
delanocamposs edee2c8
verbose = 1
delanocamposs d62af54
testing new z prop wheel-based logic
delanocamposs 8ed52d9
more verbose print statements
delanocamposs e84f5c7
printout changes
delanocamposs 2c29220
fix initial covariances
delanocamposs 4b648c6
kSlope point resolution
delanocamposs 93259fa
previous propagation residuals
delanocamposs d9dbe79
latest
delanocamposs 63e041e
reverting back to using kSlope in measurement space for residual
delanocamposs 1c15a25
tests
delanocamposs 13ff919
testing
delanocamposs 629bb34
testing
delanocamposs 2172930
updates
delanocamposs 3bc97b4
add new members and accessors for new gain values
delanocamposs 66d9d59
fixed point arithmetic for zNew and kSlope. updateLUT function pulls …
delanocamposs fc06842
packed gain theta view LUT root file
delanocamposs a04c22f
small cleanup
delanocamposs a40c085
cleanup printouts
delanocamposs 51bb165
fix wraparound bug
delanocamposs 044560b
5D vertex constraint offline/LUT updates
delanocamposs 2992c40
kSlopeNew fixed point arithmetic computation
delanocamposs 3b34903
Barrel Stub Processor cleanup
delanocamposs b3e632e
update ThetaGain function
delanocamposs b6abb60
add phi bitmask split on certain theta gains. updated theta gain root…
delanocamposs 0dda464
new smoothed Theta Gain LUT
delanocamposs 6053c53
Track functionality to hold z, kSlope at Muon station 1 and Vertex. U…
delanocamposs 51afa8f
KMTFPattern saves granular eta instead of coarse. Also saved z, kSlope.
delanocamposs c69d405
fix prinouts for KMTFPatterns
delanocamposs 655f3b8
SetPtEtaPhiDispl now stores etaDisp
delanocamposs 7c6a54a
z, kSlope saturation changes
delanocamposs 94cacf4
saturate KSlope properly and add edm::LogWarnings
delanocamposs 1a30c37
provide z0, new eta to SAMuon
delanocamposs 6d0b693
update LSB comment
delanocamposs 1f016b7
LogInfo
delanocamposs e591530
add back srcDT, srcDTTheta
delanocamposs f704d8b
Merged KMTF-upgrade-develop from repository delanocamposs with cms-me…
2872974
Merged Kalman_26 from repository zhenbinwu with cms-merge-topic
6782bb3
Code format
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is this the intended location of this file or is it planned to live elsewhere like cms-data? |
Binary file not shown.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only one new class version should be added. Very likely you also need to rebase on top of 20_0_0_pre1 (hopefully available early next week) because the checksum for version 3 will be changed for that.