a-b-street/abstreet

Smaller files: don't serialize turn geometry?

Open

#94 aberto em 24 de jun. de 2020

Ver no GitHub
 (5 comments) (1 reaction) (0 assignees)Rust (256 forks)batch import
good first issue

Métricas do repositório

Stars
 (6.779 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Explore a serialização de PolyLine em map model/src/turn.rs. Meça o impacto no tamanho usando serialized size bytes em map.rs. Implemente a omissão da serialização do campo PolyLine e regenere o em Map::new. Teste as alterações no tamanho do arquivo e no tempo de carregamento.
Pilha de tecnologia
rust
Domain
performancedata
Tipo Issue
Performance
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rust programmingSerde serializationmap model structure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60