maplibre/martin

Add more validations to .mbtiles detection

Aperta

#741 aperta il 3 lug 2023

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (0 assegnatari)Rust (377 fork)auto 404
good first issuehelp wantedmbtilesrust

Metriche repository

Star
 (3761 stelle)
Metriche merge PR
 (Merge medio 2g 16h) (78 PR mergiate in 30 g)

Descrizione

As part of mbtiles type detection, we should catch some common issues:

  • detect if tiles (simple) or map (deduplicated) tables have an index by z,x,y columns
  • ???

In case of a problem, use a warn! macro to print a warning, unless we are certain the issue makes it impossible to use a file, in which case it should be appropriate error (add it to the error enum).

A bigger discussion is if mbtiles should stop using logging, and return some Vec<MbtilesWarning> if non-critical issues are detected.

Guida contributor

Direzione di ricerca
Esamina il codice di rilevamento in mbtiles.rs, aggiungi controlli per la presenza di indici sulle colonne z,x,y nelle tabelle tiles o map. Usa warn! per problemi non critici, errore per problemi fatali. Considera l'uso di Vec<MbtilesWarning> come tipo di ritorno per gli avvisi.
Tech stack
rust
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75