Incorrect shape assertion in test_decode_prefixes_device_handling
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 52/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- python
- Área
- machine-learning, testing-qa
Línea de trabajo
Abre test/test_nn_objectives.py e inspecciona test_decode_prefixes_device_handling alrededor de la línea 491, incluida la forma esperada de decode. Ejecuta el test en un sistema con CUDA y actualiza la assertion para que coincida con el orden documentado (batch, n_prefixes, d_model).
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Bug Description
The test test_decode_prefixes_device_handling in tests/test_nn_objectives.py has an incorrect shape assertion on line 491.
Current Code
f_x = torch.randn(2, 32).cuda()
prefixes = torch.tensor([8, 16, 32])
x_hats = sae.decode(f_x, prefixes=prefixes)
assert x_hats.shape == (3, 2, 16) # BUG: Wrong shape!
Expected Behavior
The decode method returns tensors with shape (batch, n_prefixes, d_model). Given:
- batch_size = 2
- n_prefixes = 3
- d_model = 16
The correct shape should be (2, 3, 16) not (3, 2, 16).
Impact
This test is currently skipped on systems without CUDA, so the bug hasn't been caught by CI. However, on CUDA-enabled systems, this test would fail.
Fix
Change line 491 from:
assert x_hats.shape == (3, 2, 16)
to:
assert x_hats.shape == (2, 3, 16)
- Lenguaje dominante
- Python
- Estrellas
- 67
- Forks
- 12
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
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 Imageomics/saev
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
Imageomics/saev#30 · 3 comentarios ·
-
performance
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Imageomics/saev#26 · 1 reacción ·
-
Bug: BatchLimiter incorrectly counts samplesQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
Imageomics/saev#17 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
Imageomics/saev#11 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Imageomics/saev#10 ·
Todos los issues de Imageomics/saev
Issues similares
-
No test checks that scheduled workflows declare minimal permissions, concurrency and a timeoutAbiertoci enhancement testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
area/config comp/tools P2 platform/telegram type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
NousResearch/hermes-agent#145412 ·
Los mantenedores suelen responder en 1 día
-
[oblt-aw][autodoc] Update docs for security org scaffolding and Docs AI menu permission changesPosiblemente ocupada @github-actions la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
curriculum documentation quality
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
githubnext/gh-aw-workshop#5039 ·
Los mantenedores suelen responder en 1 día
-
Community project: hosted MOSS-Image studio with a visual JSON layout builder (no ComfyUI needed)Abierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 65/100
OpenMOSS/MOSS-Image#27 · 1 comentario ·
Los mantenedores suelen responder en 1 día