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.

Sign up for GitHub

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

Jump to bottom

updating and merging dev-align3 #425

Draft
wants to merge 357 commits into
base: development
Choose a base branch
from
Draft

updating and merging dev-align3 #425

wants to merge 357 commits into from

Conversation

raffaelladevita
Copy link
Collaborator

No description provided.

sebouh137 and others added 30 commits April 26, 2021 13:17
@sebouh137
added align.json from the dev-align2 branch
a3c9f1a
@sebouh137
removed shift in cvt prealignment
590177a
@sebouh137
Fixed bug where the run number was being set to the actual run number
a59edd9 
(instead of staying at 11).  Also minor stuff like using LCZ strip
@zieglerv
Added banks for alignment.
fc88634
@zieglerv
Added banks for alignment.
9d23d17
@zieglerv
Added banks for alignment.
3935071
@sebouh137
some cleanup before merging with TracTools-NewBanks
1525774
@sebouh137
Merge remote-tracking branch 'origin/TracTools-AddBanks' into dev-align3
95816a2 
allows pre-alignment to obtain some of the information directly from the recon, instead of recalculating a lot of information
@baltzell @raffaelladevita
enable github actions, to replace Travis
184accc
@raffaelladevita
yaml cleanup
ca51d43
@raffaelladevita
resolving conflict with cherry-pick
d386d06
@raffaelladevita
another yaml cleanup
4c7035b
@raffaelladevita
another yaml cleanup
4f802d1
@raffaelladevita
yaml: fixed error in data.yaml
67020a8
@raffaelladevita
disable TravisCI
0a9c037
@baltzell @raffaelladevita
remove defunct yamls
63f693c
@baltzell @raffaelladevita
rename eb yaml
fa42176
@baltzell @raffaelladevita
rename ambiguous yaml
a7dc0ae
@baltzell @raffaelladevita
replace Travis with github action badge
6850406
@raffaelladevita @baltzell
Maven blocking (#698)
31fc8de 
* using https for freehep

* one more attempt

* trying via mirror

* trying via mirror

* cleanup

Co-authored-by: Nathan Baltzell <baltzell@gmx.com>
@zieglerv
Added banks output fixes.
776a9a0
@zieglerv
Merge branch 'TracTools-AddBanks' of https://github.com/JeffersonLab/…
696b6fd 
…clas12-offline-software into TracTools-AddBanks
@sebouh137
Merge remote-tracking branch 'origin/TracTools-AddBanks' into dev-align3
db16e49
@sebouh137
use new banks in pre-align
d21d548
@zieglerv
Added banks output fixes.
0e39556
@sebouh137
Merge remote-tracking branch 'origin/TracTools-AddBanks' into dev-align3
05d9557
@zieglerv
Fix for cosmic tracks reconstruction.
100b2f0
@heddle
modified the swimToRho uniform step size swimmer
11c7410
@zieglerv
Fix for l,s,n alignment support vectors.
003f4ee
@sebouh137
Merge remote-tracking branch 'origin/TracTools-AddBanks' into dev-align3
1c28b35
sebouh137 and others added 28 commits January 18, 2022 01:14
@sebouh137
modified RecoBankReader.java to allow compilation
e981a1d
@sebouh137
Merge branch 'dev-align3' of https://github.com/JeffersonLab/clas12-o…
88fabd9 
…ffline-software into dev-align3
@sebouh137
modified CVTAlignment.java to allow compilation
77020cf
@sebouh137
Merge branch 'dev-align3' of https://github.com/JeffersonLab/clas12-o…
296d180 
…ffline-software into dev-align3
@raffaelladevita
Disabled reseeding when excludeLayers option is chosen for cosmics. T…
c0ced70 
…he layer exclusion will happen in the KF
@raffaelladevita
clas-tracking cleanups: removed commented code, fixed warnings, ...
31ed42b
@raffaelladevita
CVT package cleanup: make syntax more uniform, address warnings, remo…
62227fb 
…ve obsolete/unused methods, removed unused constants moved Measurements class to new folder, changed TrackCandListFinder to StraightTrackCandListFinder since it is used only for straight tracks
@sebouh137
merged with origin/TracTools-Test3Dev
c9cd218
@sebouh137
fixed things that broke in the merge
a0977ac
@sebouh137
fixed issue with RecoBankReader (units for xb and yb)
9e1f7da
@raffaelladevita
Changed default setting to include beam-spot cnstraint on seeding onl…
711660a 
…y and 5 KF iterations
@sebouh137
Merge remote-tracking branch 'origin/TracTools-Test5Dev' into dev-align3
5d9fae6
@sebouh137
removed line to set p=1 for straight tracks
498c432
@sebouh137
added functionality to do get residuals from curved tracks in alignme…
c3c6593 
…nt. (using curved tracks in alignment still is not functional yet.)
@sebouh137
fixed a bug in the previous commit
41216be
@raffaelladevita
Merged development
d251706
@raffaelladevita
Merge branch 'development' into dev-align44
9c9153e
@raffaelladevita
merged development into dev-align3, switched cvt alignment service to…
a22f4d9 
… use existing RecoBankReader class, fixed issues with line-cylinder intersection in straight track KF transport
@raffaelladevita
passing bank names to RecoBankReader methods for handling alignment w…
37bdd01 
…ith both tracks from target and cosmics
@raffaelladevita
revert cvt alignment to use original bank reader
76b6f39
@raffaelladevita
fixing some missing units conversion
c3c3ce4
@raffaelladevita
set straight tracks momentum to 100 GeV for alignment
20f22e6
@raffaelladevita
add checks on NaNs for cosmic tracks
eb4d408
@raffaelladevita
restored 1 GeV momentum for straight tracks
94a5b9b
@raffaelladevita
forcing magnetic field to be zero for cosmics, fix to phi->strip numb…
c6906ec 
…er of BMT-Z
@raffaelladevita
cvt alignment code cleanup and switch to ejml
e6bd110
@raffaelladevita
switching again straight track momentum to 100 GeV
7d01474
@raffaelladevita
merged development
2e921fc
@c-dilks c-dilks mentioned this pull request Jan 13, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@raffaelladevita @sebouh137 @zieglerv @baltzell @heddle