simvue-io / simvue-io/python-api
Grid.new fails if 1D array provided
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 4
- Forks
- 1
- Merge medio
- 5 d 8 h
- PR fusionados (30 d)
- 4
Descripción
Description of Bug
If trying to create a grid for a 2D metric by providing a 1D array of ticks, it fails validation.
Steps To Reproduce
Code to reproduce the behavior:
import simvue
import numpy
with simvue.Run() as run:
run.init("test")
run.assign_metric_to_grid(metric_name="test_metric_2d", axes_ticks=numpy.arange(10), axes_labels=['x'])
for i in range(10):
run.log_metrics({"test_metric_2d": numpy.random.rand(10)}, time=i, step=i)
Expected behavior
Grid added correctly
Actual Behaviour
Traceback (most recent call last):
File "/home/wk9874/Documents/simvue/python-api/test_retrieval.py", line 6, in <module>
run.assign_metric_to_grid(metric_name="test_metric_2d", axes_ticks=numpy.arange(10), axes_labels=['x'])
File "/home/wk9874/Documents/simvue/python-api/simvue/utilities.py", line 260, in wrapper
return class_func(self, *args, **kwargs)
File "/home/wk9874/Documents/simvue/python-api/simvue/run.py", line 105, in _wrapper
return _function(self, *args, **kwargs)
File "/home/wk9874/Documents/simvue/python-api/simvue/utilities.py", line 300, in wrapper
raise RuntimeError(error_str)
RuntimeError: `Grid.new` Validation:
╒═════════╤═════════════╤══════════════════╤══════════════════════════════╕
│ Input │ Location │ Type │ Message │
╞═════════╪═════════════╪══════════════════╪══════════════════════════════╡
│ 0 │ ['grid', 0] │ int != list_type │ Input should be a valid list │
├─────────┼─────────────┼──────────────────┼──────────────────────────────┤
│ 1 │ ['grid', 1] │ int != list_type │ Input should be a valid list │
├─────────┼─────────────┼──────────────────┼──────────────────────────────┤
│ 2 │ ['grid', 2] │ int != list_type │ Input should be a valid list │
├─────────┼─────────────┼──────────────────┼──────────────────────────────┤
│ 3 │ ['grid', 3] │ int != list_type │ Input should be a valid list │
├─────────┼─────────────┼──────────────────┼──────────────────────────────┤
│ 4 │ ['grid', 4] │ int != list_type │ Input should be a valid list │
├─────────┼─────────────┼──────────────────┼──────────────────────────────┤
│ 5 │ ['grid', 5] │ int != list_type │ Input should be a valid list │
├─────────┼─────────────┼──────────────────┼──────────────────────────────┤
│ 6 │ ['grid', 6] │ int != list_type │ Input should be a valid list │
├─────────┼─────────────┼──────────────────┼──────────────────────────────┤
│ 7 │ ['grid', 7] │ int != list_type │ Input should be a valid list │
├─────────┼─────────────┼──────────────────┼──────────────────────────────┤
│ 8 │ ['grid', 8] │ int != list_type │ Input should be a valid list │
├─────────┼─────────────┼──────────────────┼──────────────────────────────┤
│ 9 │ ['grid', 9] │ int != list_type │ Input should be a valid list │
╘═════════╧═════════════╧══════════════════╧══════════════════════════════╛
Screenshots
If applicable, add screenshots to help explain your problem.
Setup
- OS: [e.g. Windows 11]
- Python version [e.g. v3.13.3]
Additional context
Add any other context about the problem here.
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.
Línea de trabajo
Comienza con la llamada assign_metric_to_grid en simvue/run.py y los wrappers de validación en simvue/utilities.py; después, reproduce el fallo usando el ejemplo de NumPy proporcionado. Comprueba cómo Grid.new valida la entrada de la cuadrícula. Se considera terminado cuando un array axes_ticks 1D con una sola etiqueta crea la cuadrícula correctamente sin el error de validación indicado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- numpy, python
- Área
- api
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 42/100