test_image_reshape.py: warp/undistort/interp2d tests are empty stubs
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 72/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Tranquilo
- Stack tecnológico
- python
- Área
- computer-vision, testing
Línea de trabajo
Empieza en tests/test_image_reshape.py y compara los stubs vacíos de test_warp, test_warp_affine, test_warp_perspective, test_undistort y test_interp2d con los métodos correspondientes de ImageReshape.py. Usa las entradas y transformaciones sintéticas sugeridas en el issue y añade aserciones basadas en tolerancias para la corrección geométrica. Se considera terminado cuando cada método tiene cobertura real sin depender de la igualdad exacta de píxeles entre versiones de OpenCV.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Found 2026-08-03 while auditing warpAffine/warpPerspective/remap-based methods for OpenCV 4/5 numeric-drift issues (see #44). `tests/test_image_reshape.py`'s `test_warp`, `test_warp_affine`, `test_warp_perspective`, `test_undistort`, and `test_interp2d` are all empty stubs (`# Add test cases for the decimate method` / `pass`) -- zero real assertions, for five distinct methods in `ImageReshape.py` that wrap `cv2.warpAffine`/`cv2.warpPerspective`/`cv2.remap`/`cv2.undistort`.
`test_rotate` (same file) does have real coverage and was specifically checked against a real OpenCV 5.0.0.93 install as part of #44 -- its one exact-equality assertion (`rotate(0)`, an identity transform) is confirmed safe on both versions, no drift. But the five methods above have no coverage at all to check.
Fix
Write real tests for each: a known input + expected transform (e.g. `warp_affine` with a translation matrix, `undistort` with known distortion coefficients on a synthetic pattern), asserting geometric correctness with a tolerance rather than exact pixel equality -- OpenCV's own migration notes describe revised interpolation between versions 4 and 5, so any assertion here should tolerate small numeric drift by construction, not just by accident.
- Lenguaje dominante
- Python
- Estrellas
- 220
- Forks
- 30
- Merge medio
- 12 d 23 h
- PR fusionados (30 d)
- 5
Preparar el entorno
Aún no hemos revisado los archivos de configuración de este proyecto. Empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de petercorke/machinevision-toolbox-python
-
tech-debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
tech-debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
-
tech-debt
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
-
tech-debt
Dificultad 3/5 1-2 días Aptitud para principiantes 76/100
-
tech-debt
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
Todos los issues de petercorke/machinevision-toolbox-python
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
qgis/QGIS-Plugins-Website#459 ·
-
bug severity:medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
bot-found bug priority: P3
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
madenvel/KalinkaPlayer#179 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
ls1intum/edutelligence#1098 ·
Los mantenedores suelen responder en 1 día