acts-project / acts-project/acts

`Grid` and `MultiAxis` related naming

Aperta
#5,601 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Stale
Lingua principale
C++
Stelle
131
Fork
276
Merge medio
3g 13h
PR unite (30g)
112

Descrizione

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`

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.