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

ci: Monitor memory in physmon jobs #2268

Merged
merged 12 commits into from
Jul 6, 2023

Conversation

paulgessinger
Copy link
Member

No description provided.

@paulgessinger paulgessinger added the 🚧 WIP Work-in-progress label Jul 4, 2023
@paulgessinger paulgessinger added this to the next milestone Jul 4, 2023
@github-actions github-actions bot added the Infrastructure Changes to build tools, continous integration, ... label Jul 4, 2023
@github-actions
Copy link

github-actions bot commented Jul 4, 2023

📊 Physics performance monitoring for ef7c814

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

@codecov
Copy link

codecov bot commented Jul 4, 2023

Codecov Report

Merging #2268 (ef928e3) into main (0eebf3b) will not change coverage.
The diff coverage is n/a.

❗ Current head ef928e3 differs from pull request most recent head ef7c814. Consider uploading reports for the commit ef7c814 to get more accurate results

@@           Coverage Diff           @@
##             main    #2268   +/-   ##
=======================================
  Coverage   49.27%   49.27%           
=======================================
  Files         450      450           
  Lines       25410    25410           
  Branches    11724    11724           
=======================================
  Hits        12521    12521           
  Misses       4556     4556           
  Partials     8333     8333           

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

@paulgessinger paulgessinger removed the 🚧 WIP Work-in-progress label Jul 6, 2023
@paulgessinger
Copy link
Member Author

I can only really test whether the push to the other repo works on main itself. Since it doesn't crash here, can we merge it and then I fix any issues with shipping the results to the other repo directly on main?

Copy link
Contributor

@andiwand andiwand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fancy! looking forward to see some plots

@kodiakhq kodiakhq bot removed the automerge label Jul 6, 2023
@kodiakhq
Copy link
Contributor

kodiakhq bot commented Jul 6, 2023

This PR currently has a merge conflict. Please resolve this and then re-add the automerge label.

@kodiakhq kodiakhq bot merged commit 8288d1a into acts-project:main Jul 6, 2023
@github-actions github-actions bot removed the automerge label Jul 6, 2023
@paulgessinger paulgessinger deleted the feat/physmon-memory branch July 6, 2023 11:34
@paulgessinger paulgessinger removed this from the next milestone Jul 24, 2023
@paulgessinger paulgessinger added this to the v27.2.0 milestone Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Infrastructure Changes to build tools, continous integration, ...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants