acts-project / acts-project/acts

`Grid` and `MultiAxis` related naming

Offen
#5,601 3 Kommentare 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

While working on the `MultiAxis` implementation I stumbled over a few potentially suboptimal names

1. `global bin` and `local bins`
- `bin` is actually an `index` - we could call it that way
- `global` / `local` is a loaded term in ACTS and usually refers to different coordinate systems
- my suggestion here would be something like `flat index` or `linear index` and `multi index`
2. for `Axis` we use `getX` while for `Grid` we use `x`
3. `lower left edge` / `upper right edge`
- I believe this should rather be `corner`
4. `position` vs `point`

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

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