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

fix: Protection against v == 0 in template_switch #1707

Merged
merged 4 commits into from
Dec 9, 2022

Conversation

CarloVarni
Copy link
Collaborator

@CarloVarni CarloVarni commented Dec 5, 2022

Add a check in template_switch methods in case v == 0 (i.e. no measurements) and add corresponding error message

@codecov
Copy link

codecov bot commented Dec 9, 2022

Codecov Report

Merging #1707 (125c751) into main (6045480) will increase coverage by 0.06%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1707      +/-   ##
==========================================
+ Coverage   49.21%   49.27%   +0.06%     
==========================================
  Files         398      398              
  Lines       21790    21763      -27     
  Branches     9899     9874      -25     
==========================================
  Hits        10724    10724              
+ Misses       4203     4174      -29     
- Partials     6863     6865       +2     
Impacted Files Coverage Δ
Core/include/Acts/Utilities/Helpers.hpp 65.55% <0.00%> (-3.05%) ⬇️
Core/src/Geometry/CylinderVolumeBuilder.cpp 27.67% <0.00%> (+3.16%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@paulgessinger paulgessinger added this to the next milestone Dec 9, 2022
@kodiakhq kodiakhq bot merged commit ce28f69 into acts-project:main Dec 9, 2022
@github-actions github-actions bot removed the automerge label Dec 9, 2022
@github-actions
Copy link

github-actions bot commented Dec 9, 2022

📊 Physics performance monitoring for 125c751

Full report
Seeding: seeded, truth estimated, orthogonal
CKF: seeded, truth smeared, truth estimated, orthogonal
IVF: seeded, truth smeared, truth estimated, orthogonal
Ambiguity resolution: seeded, orthogonal
Truth tracking
Truth tracking (GSF)

Vertexing

Vertexing vs. mu
IVF seeded

IVF truth_smeared

IVF truth_estimated

IVF orthogonal

Seeding

Seeding seeded

Seeding truth_estimated

Seeding orthogonal

CKF

CKF seeded

CKF truth_smeared

CKF truth_estimated

CKF orthogonal

Ambiguity resolution

seeded

Truth tracking (Kalman Filter)

Truth tracking

Truth tracking (GSF)

Truth tracking

@CarloVarni CarloVarni deleted the ProtectionNoMeasurements branch December 10, 2022 11:14
pbutti pushed a commit to pbutti/acts that referenced this pull request Dec 13, 2022
Add a check in `template_switch` methods in case `v == 0` (i.e. no measurements) and add corresponding error message
@paulgessinger paulgessinger modified the milestones: next, v22.0.0 Dec 21, 2022
CarloVarni added a commit to CarloVarni/acts that referenced this pull request Dec 22, 2022
Add a check in `template_switch` methods in case `v == 0` (i.e. no measurements) and add corresponding error message
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.

None yet

2 participants