Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

`_points_equal` incorrectly compares floats with `is not`

Abierto
  • ¿Algo no está bien?
#19 1 comentario 0 reacciones 0 asignados Ver en GitHub

@SashaMIT ya está trabajando en esto.

Desde el 27/9/2026.

  • #27 de @SashaMIT — abierto

Evaluación

Dificultad
1/5
Tiempo estimado
Menos de una hora
Aptitud para principiantes
45/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
python
Área
backend

Línea de trabajo

Abre sno/pyvendor/vector_tile_base/engine.py e inspecciona _points_equal alrededor de la línea 313. Reemplaza la comparación de identidad por una comparación de valores, luego ejecuta las pruebas existentes del proyecto y verifica que unas coordenadas de punto flotante iguales permitan reconocer un anillo cerrado.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

I noticed add_ring is not treating my closed ring like a closed ring:

ipdb> pt1
(19188372.37109346, -5402212.477493137)
ipdb> pt2
(19188372.37109346, -5402212.477493137)
ipdb> n
> /Users/cdestigter/checkout/sno/sno/pyvendor/vector_tile_base/engine.py(314)_points_equal()
    313         if pt1[0] is not pt2[0] or pt1[1] is not pt2[1] or (self._has_elevation and pt1[2] is not pt2[2]):
--> 314             return False
    315         return True

ipdb> pt1[0] == pt2[0]
True
ipdb> pt1[0] is pt2[0]
False

is not isn't the appropriate comparator here, it shoudl be !=. python doesn't guarantee two equal floats are the same float.

Lenguaje dominante
Python
Estrellas
57
Forks
17
Sin PR fusionados en 30 d

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de mapbox/vector-tile-base

Todos los issues de mapbox/vector-tile-base

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.