voxel51/fiftyone

[FR] Verify latitude and longitude values at creation time

Open

#2171 aperta il 14 ott 2022

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Python (400 fork)batch import
coreenhancementgood first issue

Metriche repository

Star
 (4021 star)
Metriche merge PR
 (Merge medio 3g 11h) (161 PR mergiate in 30 g)

Descrizione

Invalid lat and long coordinates in a GeoLocation field will result in the App throwing an error:

Error: Invalid LngLat latitude value: must be between -90 and 90
    at new de (http://localhost:5151/assets/index.f01320f2.js:711:66777)
    at de.convert (http://localhost:5151/assets/index.f01320f2.js:711:67409)
    at Q.extend (http://localhost:5151/assets/index.f01320f2.js:711:65535)
    at http://localhost:5151/assets/index.f01320f2.js:1790:65057
    at Array.reduce (<anonymous>)
    at bye (http://localhost:5151/assets/index.f01320f2.js:1790:65016)
    at http://localhost:5151/assets/index.f01320f2.js:1790:65695
    at Object.Gae [as useMemo] (http://localhost:5151/assets/index.f01320f2.js:34:23601)
    at Eo.useMemo (http://localhost:5151/assets/index.f01320f2.js:9:6259)
    at Tst (http://localhost:5151/assets/index.f01320f2.js:1790:65683)

It would be best to catch this error at data ingestion time.

Guida contributor

Direzione di ricerca
Il problema suggerisce di aggiungere la validazione per latitudine e longitudine durante la creazione di un campo GeoLocation. Esamina il codice di creazione dei campi in FiftyOne, probabilmente in `fiftyone/core/fields.py` o `fiftyone/core/odm.py`. Controlla come sono attualmente definiti i campi GeoLocation e da dove origina l'errore (probabilmente da Mapbox GL JS nel frontend). L'obiettivo è aggiungere la validazione lato Python prima che i dati raggiungano il database. Rivedi i modelli di validazione esistenti nella codebase e aggiungi un controllo nel metodo `validate` del campo `GeoLocation` o simile.
Tech stack
python
Dominio
databackend
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonFiftyOne
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70