acts-project / acts-project/acts

Implement `EllipseBounds::closestPoint` after #3990

Offen
#4,478 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Stale
Vorherrschende Sprache
C++
Sterne
131
Forks
276
Ø Merge
3 T. 13 Std.
Gemergte PRs (30 T.)
112

Beschreibung

https://github.com/acts-project/acts/pull/3990 changed the boundary tolerance to consistently use `closestPoint` for the computation. `EllipseBounds` are a bit special here and have been left out for now because there are a few open questions:
- Are the ellipse axes really defined as local vectors? To me it seems much easier to just provide the radii. This would also be less error prone.
- There is also an average phi angle in the bounds parameters - seems like this duplicates the vectors mentioned above?
- Implementing `closestPoint` seems straight forward for euclidean distance as soon as the points above are clarified.
- Implementing `closestPoint` using the mahalanobis distance is more complicated. But I think the ellipse can be transformed after decomposing the metric into another ellipse?

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.