uber/h3

Inline _isBaseCellPentagon

Aperta

#984 aperta il 9 mar 2025

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)C (585 fork)github user discovery
help wanted

Metriche repository

Star
 (6310 stelle)
Metriche merge PR
 (Merge medio 7g 19h) (17 PR mergiate in 30 g)

Descrizione

This is a follow-up for https://github.com/uber/h3/pull/968

As discussed in https://github.com/uber/h3/pull/968, it would be nice to inline _isBaseCellPentagon for use in isValidCell (and potentially other spots in the code base).

I couldn't get this working in https://github.com/uber/h3/pull/968 after trying a mixture of inline and extern, so making this ticket to follow up and revisit.

Some notes:

  • maybe benchmark the array of structs vs stuct of arrays in baseCellData

Guida contributor

Direzione di ricerca
Esamina l'attuale funzione isBaseCellPentagon e il suo utilizzo in isValidCell. Studia il layout della struct baseCellData e prova a inline con diverse combinazioni di inline ed extern. Esegui benchmark dell'array di struct rispetto alla struct di array per misurare l'impatto sulle prestazioni.
Tech stack
c
Dominio
backendapi
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C programmingunderstanding of H3 base cell data structuresbenchmarking
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60