Treat corrupt-shape state files as missing in `load_state`
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 85/100
Direzione di ricerca
In overturemaps/state.py, leggi load_state e la sua gestione attuale delle eccezioni. Aggiungi test per un file di stato JSON contenente una lista e per un valore di backend sconosciuto, come descritto nell’issue. Il lavoro è completato quando entrambe le strutture non valide restituiscono None senza traceback e la docstring descrive sia lo stato non valido sia i file mancanti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Context
A state file with valid JSON but the wrong shape (e.g. a top-level list, or an unknown backend value) makes overturemaps releases check crash with an unhandled AttributeError or ValueError traceback. Missing files, invalid JSON and missing keys already fall back to "no state"; wrong-shape files should do the same.
Proposed change
In state.py:
- Extend the except clause in
load_stateto also catchAttributeError,TypeErrorandValueErrorand returnNone. - Update the docstring to say
Noneis returned for invalid state, not just missing files. - Add tests for a JSON list state file and an unknown
backendvalue.
- Lingua principale
- Python
- Stelle
- 280
- Fork
- 48
- Merge medio
- 4h 28m
- PR unite (30g)
- 16
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 OvertureMaps/overturemaps-py
-
download by country codeApertaexternal
Difficoltà 4/5 3-5 giorni Idoneità per principianti 47/100
OvertureMaps/overturemaps-py#148 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Include theme/type in outputForse di nuovo libera @danabauer l’ha presa 283 giorni fa e non c’è nessuna pull request aperta. Apertaenhancement
OvertureMaps/overturemaps-py#24 · 1 reazione · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Allow selecting specific columnsForse di nuovo libera @danabauer l’ha presa 283 giorni fa e non c’è nessuna pull request aperta. Apertaenhancement
OvertureMaps/overturemaps-py#13 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Use pyogrioForse di nuovo libera @danabauer l’ha presa 283 giorni fa e non c’è nessuna pull request aperta. Apertaenhancement
OvertureMaps/overturemaps-py#9 · 2 commenti · 1 reazione · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di OvertureMaps/overturemaps-py
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
FAIRmat-NFDI/pynxtools#861 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
NVIDIA/nsight-python#88 ·
I maintainer di solito rispondono entro 1 giorno
-
Criteria can pass when the deliverable is missingForse già presa @clementtisseau l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
harveyai/harvey-labs#217 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
priority/medium roadmap/v1.1.1 stakeholder/rapr type/bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
databrickslabs/ontos#1420 ·
I maintainer di solito rispondono entro 3 giorni