a-b-street/abstreet

Smaller files: don't serialize turn geometry?

Open

#94 aperta il 24 giu 2020

Vedi su GitHub
 (5 commenti) (1 reazione) (0 assegnatari)Rust (256 fork)batch import
good first issue

Metriche repository

Star
 (6779 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Not sure this will work, but it's a good little exploratory project. map_model/src/turn.rs stores geom: PolyLine. Last time I used the serialized_size_bytes stuff in map.rs, these polylines constituted a huge part of the binary map files. The release size is slowly growing, so it'd be nice to trim this down.

Maybe we could skip serializing the PolyLine, then in Map::new after deserializing, go fill it in again. Should measure the impact on file sizes and initial map loading time.

Guida contributor

Direzione di ricerca
Esplora la serializzazione di PolyLine in map model/src/turn.rs. Misura l'impatto sulle dimensioni utilizzando serialized size bytes in map.rs. Implementa il salto della serializzazione del campo PolyLine e rigeneralo in Map::new. Testa le variazioni delle dimensioni del file e del tempo di caricamento.
Tech stack
rust
Dominio
performancedata
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust programmingSerde serializationmap model structure
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60