-
Notifications
You must be signed in to change notification settings - Fork 68
Fwd picodst integration #639
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
Merged
Merged
Changes from 38 commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
227abe1
nill
jdbrice 3da2dba
Add StPicoFwdTrack
jdbrice 3f13490
Add initial StPicoFcsCluster implementation
jdbrice c90af9a
Merge branch 'star-bnl:main' into fwd-picodst
jdbrice e6daca5
Add StPicoFcsHit, fix minor typos
jdbrice d018a1c
Merge branch 'star-bnl:main' into fwd-picodst
jdbrice f586a7b
Merge branch 'fwd-picodst' of github.com:jdbrice/star-sw-1 into fwd-p…
jdbrice 8f24db3
Updates to StPicoFwdTrack
jdbrice babd08f
merge in Updated FcsHit codes
4f2d19e
Merge branch 'star-bnl:main' into fwd-picodst
jdbrice d188093
Update from akio
jdbrice 61525da
push FWD related branches to the end to maintain backward compat
jdbrice 0376478
Apply suggestions from code review
plexoos 9da379b
fix typo, assigning id sets index instead, add DCA / global / vtx ind…
jdbrice fc6ab8d
Merge branch 'main' into fwd-picodst
jdbrice 70080dc
assign missing data on FwdTracks, make cluster<->track association
jdbrice 4bb3b18
logic to fill matches between fcs clusters and fwd tracks
jdbrice 6437f2e
add missing mGeoCache in StFwdTrackMaker, why is it missing?
jdbrice 8855ce7
signed/unsigned compare error. protect against fcs clusters not found…
jdbrice bd16fb0
notes on what needs to be added/updated in comments
jdbrice 32bf44f
Merge branch 'main' into fwd-picodst
jdbrice 5ebc238
add pVal to PicoFwdTrack and setup for rebuilding in mem the FcsClust…
jdbrice 0a2aa7b
Add PicoFwdTrack projections to HCal and ECAl
jdbrice 3822697
fix shebang in viz2.C
jdbrice 042659f
Updates to StPicoDst to incorporate FwdTrack types and FwdVertex
jdbrice eec2858
Add PicoVtxless option & prevent crash for null primary vertex
jdbrice 70d893e
Merge branch 'main' into fwd-picodst
jdbrice 11e42c3
Updates to StEvent/StFwdTrack.h for passing global track index throug…
jdbrice 3bcb400
Merge branch 'main' into fwd-picodst
plexoos 894d993
Adress comments from Dmitri
jdbrice 543c97a
Protect StPicoUtilities calculateRefMult against nullptr MuDst primar…
jdbrice c28d050
More protection in StPicoDstMaker for events without primary vertices…
jdbrice 791c93e
initialize variables in ctors
jdbrice 1005758
Merge branch 'main' into fwd-picodst
jdbrice d1d5021
Merge branch 'main' into fwd-picodst
jdbrice 4609c8b
Add BFC option for PicoVtxless
jdbrice 992e084
another safety check for picodst when there are no primary vertices/t…
jdbrice 0a8d839
Grigories comments
jdbrice 6599f12
Update StPicoVertex Print
jdbrice ad50057
Update StRoot/StPicoEvent/StPicoFwdVertex.h
jdbrice 9a76c4c
Update StRoot/StPicoEvent/StPicoFwdVertex.h
jdbrice d359006
Update StRoot/StPicoEvent/StPicoFcsHit.cxx
jdbrice 515c3ee
typo fix
jdbrice 77ee867
Log error for cluster mismatch only once per event
jdbrice 9949d29
Remove redundent test on #pv when only #primary tracks is needed
jdbrice edef3b5
Dont skip StPicoDstMaker::fillTracks when zero primarytracks
jdbrice b857d1c
fix typo, inverted logic on nPrimarys
jdbrice 6a5a064
Pass new "PicoVtxless" option to the PicoDst Maker
jdbrice e2c9144
Merge branch 'main' into fwd-picodst
nigmatkulov 2786fd9
Merge branch 'main' into fwd-picodst
nigmatkulov e184fff
Improve fillFcsClusters() for cases of no FCS collection
genevb e5999c8
Improve fillFcsHits() for cases of no FCS collection
genevb 3b468ef
Improve fillFwdTracks() for cases of no FWD track collection
genevb 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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.