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: comment_template bugs for seeding monitoring #1683

Merged
merged 8 commits into from
Nov 23, 2022

Conversation

LuisFelipeCoelho
Copy link
Member

This PR fixes some bugs from #1665 seeding performance monitoring

@LuisFelipeCoelho LuisFelipeCoelho added Bug Something isn't working Impact - Minor Nuissance bug and/or affects only a single module labels Nov 22, 2022
@LuisFelipeCoelho LuisFelipeCoelho added this to the next milestone Nov 22, 2022
@LuisFelipeCoelho LuisFelipeCoelho added the 🚧 WIP Work-in-progress label Nov 22, 2022
@codecov
Copy link

codecov bot commented Nov 22, 2022

Codecov Report

Merging #1683 (022a70c) into main (3f799e9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1683   +/-   ##
=======================================
  Coverage   48.96%   48.96%           
=======================================
  Files         397      397           
  Lines       21505    21505           
  Branches     9802     9802           
=======================================
  Hits        10530    10530           
  Misses       4179     4179           
  Partials     6796     6796           

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

@github-actions
Copy link

github-actions bot commented Nov 22, 2022

📊 Physics performance monitoring for 022a70c

🟥 ERROR The result has missing elements!
This is likely a physmon job failure

Full report
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_smeared

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

@LuisFelipeCoelho LuisFelipeCoelho marked this pull request as ready for review November 22, 2022 11:32
@LuisFelipeCoelho LuisFelipeCoelho removed the 🚧 WIP Work-in-progress label Nov 22, 2022
@kodiakhq kodiakhq bot merged commit 89c62ea into acts-project:main Nov 23, 2022
@LuisFelipeCoelho LuisFelipeCoelho deleted the seeding-mon-bu branch November 23, 2022 09:37
@LuisFelipeCoelho LuisFelipeCoelho restored the seeding-mon-bu branch November 24, 2022 16:39
kodiakhq bot pushed a commit that referenced this pull request Nov 25, 2022
This PR removes the nonexistent truth_smeared seeding plots from comment_template.md
Sorry, this is something I forgot to remove in PR #1683 😅
pbutti pushed a commit to pbutti/acts that referenced this pull request Dec 13, 2022
This PR fixes some bugs from acts-project#1665 seeding performance monitoring
pbutti pushed a commit to pbutti/acts that referenced this pull request Dec 13, 2022
…s-project#1694)

This PR removes the nonexistent truth_smeared seeding plots from comment_template.md
Sorry, this is something I forgot to remove in PR acts-project#1683 😅
@paulgessinger paulgessinger modified the milestones: next, v22.0.0 Dec 21, 2022
CarloVarni pushed a commit to CarloVarni/acts that referenced this pull request Dec 22, 2022
This PR fixes some bugs from acts-project#1665 seeding performance monitoring
CarloVarni pushed a commit to CarloVarni/acts that referenced this pull request Dec 22, 2022
…s-project#1694)

This PR removes the nonexistent truth_smeared seeding plots from comment_template.md
Sorry, this is something I forgot to remove in PR acts-project#1683 😅
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Impact - Minor Nuissance bug and/or affects only a single module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants