voxel51/fiftyone

[FR] Verify latitude and longitude values at creation time

Aberta

#2.171 aberto em 14 de out. de 2022

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)Python (400 forks)batch import
coreenhancementgood first issue

Métricas do repositório

Stars
 (4.021 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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.

Guia do colaborador

Direção de pesquisa
A questão sugere adicionar validação para latitude e longitude ao criar um campo GeoLocation. Examine o código de criação de campos no FiftyOne, provavelmente em `fiftyone/core/fields.py` ou `fiftyone/core/odm.py`. Verifique como os campos GeoLocation são atualmente definidos e onde o erro se origina (provavelmente do Mapbox GL JS no frontend). O objetivo é adicionar validação do lado do Python antes que os dados atinjam o banco de dados. Revise os padrões de validação existentes na base de código e adicione uma verificação no método `validate` do campo `GeoLocation` ou similar.
Pilha de tecnologia
python
Domain
databackend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
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.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonFiftyOne
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70