acts icon indicating copy to clipboard operation
acts copied to clipboard

feat!: Space point implementation

Open CarloVarni opened this issue 1 year ago • 9 comments

Read-only implementation of the space point EDM, following the examples from Tracks and MTJ. The seeding infrastructures have been adapted to this, both the default and orthogonal seeding.

This also removed the following classes: InternalSpacePoint and InternalSeed

CarloVarni avatar Feb 27 '23 20:02 CarloVarni

Hi Carlo, I think this is what we want. From what I looked at the code there may be few missing pieces (to be able to build this container). And I have some technical questions/doubts.

tboldagh avatar Mar 01 '23 08:03 tboldagh

this now needs both https://github.com/acts-project/acts/pull/1948 and https://github.com/acts-project/acts/pull/1966

CarloVarni avatar Mar 20 '23 13:03 CarloVarni

This issue/PR has been automatically marked as stale because it has not had recent activity. The stale label will be removed if any interaction occurs.

stale[bot] avatar May 20 '23 14:05 stale[bot]

still a lot of work to do here, but the basic structure is in place. @paulgessinger @tboldagh can you take a look at this?

CarloVarni avatar Jun 09 '23 08:06 CarloVarni

Codecov Report

Attention: Patch coverage is 11.50895% with 346 lines in your changes missing coverage. Please review.

Project coverage is 48.71%. Comparing base (d8e46ff) to head (bad00ac). Report is 374 commits behind head on main.

Files Patch % Lines
Core/include/Acts/EventData/SpacePointData.ipp 0.00% 92 Missing :warning:
...ore/include/Acts/EventData/SpacePointContainer.ipp 0.00% 61 Missing :warning:
Core/include/Acts/Seeding/SpacePointGrid.ipp 0.00% 42 Missing :warning:
Core/include/Acts/Seeding/SeedFinder.ipp 0.00% 31 Missing :warning:
Core/include/Acts/EventData/SpacePointProxy.ipp 0.00% 30 Missing :warning:
Core/include/Acts/Seeding/SeedFilter.ipp 0.00% 20 Missing :warning:
...ore/include/Acts/EventData/SpacePointContainer.hpp 0.00% 17 Missing :warning:
...e/include/Acts/EventData/SpacePointMutableData.ipp 0.00% 13 Missing :warning:
...include/Acts/EventData/SpacePointProxyIterator.ipp 0.00% 12 Missing :warning:
Core/include/Acts/Seeding/BinnedGroupIterator.ipp 69.69% 2 Missing and 8 partials :warning:
... and 5 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1900      +/-   ##
==========================================
- Coverage   48.90%   48.71%   -0.19%     
==========================================
  Files         491      498       +7     
  Lines       28561    28691     +130     
  Branches    13487    13569      +82     
==========================================
+ Hits        13967    13978      +11     
- Misses       4861     4980     +119     
  Partials     9733     9733              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jun 28 '23 15:06 codecov[bot]

/cc-ing the experts @LuisFelipeCoelho @noemina @paulgessinger @stephenswat @tboldagh

CarloVarni avatar Jun 29 '23 08:06 CarloVarni

how do we proceed here @CarloVarni @paulgessinger ?

andiwand avatar Mar 18 '24 07:03 andiwand

I propose to still keep this open. This has gone down in the priority list. I'd like to apply possibly a couple of changes to the Seed Finder and grid creation before.

CarloVarni avatar Mar 18 '24 08:03 CarloVarni

Quality Gate Failed Quality Gate failed

Failed conditions
0.5% Coverage on New Code (required ≥ 25%)
11 New Bugs (required ≤ 0)
E Reliability Rating on New Code (required ≥ A)
0.83% Line Coverage on New Code (required ≥ 50%)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

sonarqubecloud[bot] avatar Jul 23 '24 21:07 sonarqubecloud[bot]