Older version of FMpy's method completedIntegratorStep() returns a int instead of a bool
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 64/100
Línea de trabajo
Empieza con tests/pathsim/utils/test_fmuwrapper.py, especialmente con TestFMUWrapperModelExchange.test_completed_integrator_step, y sigue el valor devuelto por FMUWrapper.completed_integrator_step. Reproduce el fallo con la configuración indicada de Windows, Python 3.12.14 y FMPy 0.3.20; compara el valor devuelto en distintas plataformas si es posible. La tarea estará terminada cuando pase la prueba y enter_event_mode sea un bool de Python.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
FMUWrapper.completed_integrator_step return arg enter_event_mode appears to be no longer a boolean but a int
Steps to Reproduce
- pathsim with latest version (commit https://github.com/pathsim/pathsim/commit/a4e824de226c5a76509d6163b3c1e33445861144)
- fmpy == 0.3.20
- python ==3.12.14
Expected Behavior
tests.pathsim.utils.test_fmuwrapper.TestFMUWrapperModelExchange.test_completed_integrator_step should not fail
Actual Behavior
tests.pathsim.utils.test_fmuwrapper.TestFMUWrapperModelExchange.test_completed_integrator_step fails with the mention AssertionError: 0 is not an instance of <class 'bool'>
Screenshots / Logs
======================================================================
FAIL: test_completed_integrator_step (tests.pathsim.utils.test_fmuwrapper.TestFMUWrapperModelExchange.test_completed_integrator_step)
Test completed integrator step notification
----------------------------------------------------------------------
Traceback (most recent call last):
File "C:\Users\pimss\Documents\workspace\pathsim\tests\pathsim\utils\test_fmuwrapper.py", line 280, in test_completed_integrator_step
self.assertIsInstance(enter_event_mode, bool)
AssertionError: 0 is not an instance of <class 'bool'>
----------------------------------------------------------------------
Ran 1208 tests in 312.165s
FAILED (failures=1, skipped=14)
print(type(enter_event_mode).__name__) returns int
Environment
- OS: Windows 11
Additional Context
I suspect this may be a windows specific error as recents PR to pathsim have not raise this error and fmpy 0.3.20 is the same since 2024-02-15
- Lenguaje dominante
- Python
- Estrellas
- 507
- Forks
- 54
- Merge medio
- 10 h 44 min
- PR fusionados (30 d)
- 9
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 pathsim/pathsim
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
-
simulink-pathsim converterAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Todos los issues de pathsim/pathsim
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 80/100
521xueweihan/HelloGitHub#4063 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Vanilla-OS/apx-gui#112 ·
-
Devstral and Voxtral: section 5 fails in us-east-1 with 400 "Model does not support audio modality"Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
aws-samples/sample-per-model-bedrock#409 ·
Los mantenedores suelen responder en 1 día
-
DuckDB 2.0 breaks to_pyarrow_batchesPosiblemente ocupada @DG47 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
ibis-project/ibis#12196 ·
Los mantenedores suelen responder en 1 día