-
Notifications
You must be signed in to change notification settings - Fork 225
feat: update to actsv45.0.0 #4272
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
Draft
osbornjd
wants to merge
55
commits into
sPHENIX-Collaboration:master
Choose a base branch
from
osbornjd:actsv45.0.0
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.
Draft
Changes from all commits
Commits
Show all changes
55 commits
Select commit
Hold shift + click to select a range
c258c79
update calibrator
osbornjd 3b875db
call wrapper tgeo builder
osbornjd 9646765
preserve makefile while testing
osbornjd e219ff4
use helper functions
osbornjd 1f11a69
abolished measurement object
osbornjd 707c3b1
update gsf apis
osbornjd bbdeda0
trackbase compiles
osbornjd 6d1c301
Merge branch 'master' of github.com:sPHENIX-Collaboration/coresoftwar…
osbornjd 0c9988a
update alignment states
osbornjd b7c4989
fix acts propagator
osbornjd 67f930d
fix geometry building API
osbornjd e6240d3
fix acts evaluator
osbornjd 9ea65df
geometry builder compiles
osbornjd a419369
makesourcelinks compiles
osbornjd a1b48d8
gsf compiles
osbornjd 8a4c141
fix include
osbornjd c547c80
add fwd decl
osbornjd d2e9a1c
add some updates for compilation
osbornjd 3804288
go back to regular makefile
osbornjd 567fda4
Return other lines from makefile to normal
osbornjd df117a7
add missing fstream include
osbornjd 100fa26
Readd mag field options
osbornjd aabd8ae
fix field map reading
osbornjd fbfe26e
add in field options
osbornjd 5e88868
add fwd decl
osbornjd 3dedddb
Get kd seeder to compile locally
osbornjd 603c32a
working through silicon seeding API changes
osbornjd a0f0087
finally get all templated changes sorted out
osbornjd c21d9d5
track fitters compile
osbornjd 2df35fc
remove deprecated library names
osbornjd d303120
remove missing include
osbornjd 3167d91
make single typename definition for new acts propagator options
osbornjd c4220c8
Merge branch 'master' of github.com:sPHENIX-Collaboration/coresoftwar…
osbornjd b2b2ba9
fix func signature
osbornjd 97a7f52
Remove acts dependency which is unneccessary
osbornjd abfd54b
Merge branch 'master' of github.com:sPHENIX-Collaboration/coresoftwar…
osbornjd d91092f
remove finalize method
osbornjd e58e15f
Remove finalize
osbornjd f5a5135
Merge branch 'master' of github.com:sPHENIX-Collaboration/coresoftwar…
osbornjd 108c24b
make sure detector does not leave scope
osbornjd a1aae44
clean up
osbornjd bd1e06b
remove deadweight
osbornjd f0a6e98
Merge branch 'master' of github.com:sPHENIX-Collaboration/coresoftwar…
osbornjd 3891538
call directed fitter constructor with proper logger
osbornjd 1e3c48c
geometry/alignment code updated
osbornjd ac93f35
update outlier finder
osbornjd c8a3b43
remove gsf from build for now, too volatile
osbornjd 0011ffa
update to v45
osbornjd 036ba65
update transform calls
osbornjd 99b9c2d
improve makefile
osbornjd 35a64f8
remove gsf from build for now
osbornjd bc93a21
update apis for several modules
osbornjd 6bc9101
update remaining apis
osbornjd 40e8aa1
update transform call
osbornjd 3364377
Merge branch 'master' of github.com:sPHENIX-Collaboration/coresoftwar…
osbornjd 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
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
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
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.
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.
Guard
fitpoint_mvtx_half_localbehindstraight_line_fit.fitpoint_mvtx_halfis only assigned in the straight-line branch, but Lines 597-598 always transform it. On the helical-fit path this is an uninitialized read and undefined behavior.🐛 Proposed fix
📝 Committable suggestion