Usage of default values in protobuf
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 48/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Ambito
- documentation
Direzione di ricerca
Inizia individuando le definizioni .proto dei campi version e extent menzionati nella issue, quindi esamina come i loro valori predefiniti sono rappresentati nella specifica delle vector tile. Il lavoro è completo quando i valori predefiniti non conformi alla specifica vengono rimossi da .proto e i valori predefiniti richiesti per version e extent vengono definiti nella specifica delle vector tile.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The google protobuf v3 spec doesn't allow you to define default values any more. In v3 defaults are what you would expect: 0 for integers, empty strings for strings, whatever has value 0 in enums.
(For details see https://developers.google.com/protocol-buffers/docs/proto3 and https://github.com/google/protobuf/blob/master/CHANGES.txt#L309).
We are using protobuf v2 currently, but to keep our options open for later switching to protobuf v3 I suggest we get rid of default values that don't correspond to the default values in the protobuf spec in the .proto and move any defaults we need into the vector tile spec.
Specifically this is the version field which has a default of 1 and the extent field which has a default of 4096.
- Lingua principale
- Nessun dato sulla lingua
- Stelle
- 1k
- Fork
- 213
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di mapbox/vector-tile-spec
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
mapbox/vector-tile-spec#157 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
mapbox/vector-tile-spec#153 ·
-
Maximum number of verticesAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
mapbox/vector-tile-spec#149 ·
-
Typo in §4.4Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 55/100
mapbox/vector-tile-spec#148 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
mapbox/vector-tile-spec#145 · 2 commenti · 1 reazione ·
Tutte le issue di mapbox/vector-tile-spec
Issue simili
-
sync-en
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
sync-en
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
I maintainer di solito rispondono entro 4 giorni
-
[oblt-aw][autodoc] Update user onboarding guide for mandatory trigger CODEOWNERS stepForse già presa @github-actions l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
I maintainer di solito rispondono entro 1 giorno
-
documentation good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
graygarbage/HermesKey#20 ·
-
8.9 to 8.10 Helm upgrade guide still says deprecated keys are removed in chart v16 (Camunda 8.11)Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100
camunda/camunda-platform-helm#7559 ·
I maintainer di solito rispondono entro 1 giorno