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

chore: Accumulated small changes #2293

Merged
merged 4 commits into from
Jul 11, 2023

Conversation

andiwand
Copy link
Contributor

I pulled this out of #2086 since it is not relevant there but I would still like to merge them. These changes are random improvements across the code base which should not have any effects on CPU or physics performance.

@github-actions github-actions bot added Component - Core Affects the Core module Infrastructure Changes to build tools, continous integration, ... Component - Examples Affects the Examples module Event Data Model Track Fitting labels Jul 11, 2023
@andiwand andiwand added this to the next milestone Jul 11, 2023
@CarloVarni CarloVarni added the 🚧 WIP Work-in-progress label Jul 11, 2023
@codecov
Copy link

codecov bot commented Jul 11, 2023

Codecov Report

Merging #2293 (3b1bb13) into main (c4ba836) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2293   +/-   ##
=======================================
  Coverage   49.27%   49.27%           
=======================================
  Files         450      450           
  Lines       25408    25408           
  Branches    11727    11727           
=======================================
  Hits        12521    12521           
  Misses       4549     4549           
  Partials     8338     8338           
Impacted Files Coverage Δ
Core/src/TrackFitting/GainMatrixSmoother.cpp 8.00% <ø> (ø)
Core/src/Material/Interactions.cpp 74.19% <100.00%> (ø)

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

@github-actions
Copy link

github-actions bot commented Jul 11, 2023

📊 Physics performance monitoring for 3b1bb13

Summary
Full report
Seeding: seeded, truth estimated, orthogonal
CKF: seeded, truth smeared, truth estimated, orthogonal
IVF: seeded, truth smeared, truth estimated, orthogonal
AMVF: 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

AMVF seeded

AMVF truth_smeared

AMVF truth_estimated

AMVF 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

@andiwand andiwand marked this pull request as ready for review July 11, 2023 14:19
@kodiakhq kodiakhq bot merged commit 617d03f into acts-project:main Jul 11, 2023
57 checks passed
@andiwand andiwand deleted the accumulated-small-changes branch July 11, 2023 15:35
@acts-project-service acts-project-service added the Fails Athena tests This PR causes a failure in the Athena tests label Jul 11, 2023
@CarloVarni CarloVarni removed the Fails Athena tests This PR causes a failure in the Athena tests label Jul 11, 2023
@paulgessinger paulgessinger modified the milestones: next, v27.2.0 Jul 24, 2023
kodiakhq bot pushed a commit that referenced this pull request Jul 24, 2023
Our full chain pulls are in a bad state. Looks like the reconstruction and simulation energy loss did not match up. This PR switches the Fatras interactions on which should bring our pulls back to standard normal distribution.

Fixes
- #1643

Blocked by
- #2157
- #2239
- #2295
- #2293
- #2294
paulgessinger pushed a commit to paulgessinger/acts that referenced this pull request Jul 24, 2023
I pulled this out of acts-project#2086 since it is not relevant there but I would still like to merge them. These changes are random improvements across the code base which should not have any effects on CPU or physics performance.
paulgessinger pushed a commit to paulgessinger/acts that referenced this pull request Jul 24, 2023
…roject#2086)

Our full chain pulls are in a bad state. Looks like the reconstruction and simulation energy loss did not match up. This PR switches the Fatras interactions on which should bring our pulls back to standard normal distribution.

Fixes
- acts-project#1643

Blocked by
- acts-project#2157
- acts-project#2239
- acts-project#2295
- acts-project#2293
- acts-project#2294
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Core Affects the Core module Component - Examples Affects the Examples module Event Data Model Infrastructure Changes to build tools, continous integration, ... Track Fitting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants