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: Boundless Portal can be constructed from JSON #2327

Merged
merged 2 commits into from
Aug 1, 2023

Conversation

paulgessinger
Copy link
Member

Also adds an assertion to the Portal constructor and the fromJson function to check if the deserialization has worked.
debar)?

Also adds an assertion to the Portal constructor and the `fromJson`
function to check if the deserialization has worked.
@paulgessinger paulgessinger added this to the next milestone Jul 27, 2023
@github-actions github-actions bot added Component - Core Affects the Core module Component - Plugins Affects one or more Plugins labels Jul 27, 2023
@andiwand andiwand self-requested a review July 27, 2023 16:11
@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #2327 (35ace5f) into main (61df945) will increase coverage by 0.00%.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##             main    #2327   +/-   ##
=======================================
  Coverage   49.67%   49.67%           
=======================================
  Files         453      453           
  Lines       25524    25526    +2     
  Branches    11701    11702    +1     
=======================================
+ Hits        12678    12679    +1     
  Misses       4571     4571           
- Partials     8275     8276    +1     
Files Changed Coverage Δ
Core/src/Detector/Portal.cpp 70.96% <33.33%> (-0.70%) ⬇️

📣 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 27, 2023

📊 Physics performance monitoring for 35ace5f

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

@kodiakhq kodiakhq bot merged commit 6276311 into acts-project:main Aug 1, 2023
53 checks passed
@github-actions github-actions bot removed the automerge label Aug 1, 2023
@paulgessinger paulgessinger modified the milestones: next, v28.1.0 Aug 7, 2023
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 - Plugins Affects one or more Plugins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants