Rework TrajectoryReporter
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Start by reading TrajectoryReporter and the report and _configure_reporter behavior described in the issue, then review the linked H5MD issue (#19) for compatibility requirements. The scope and preferred design are not settled here, so clarify the expected API before changing code. Done means the reporter supports the agreed H5MD and model-property reporting needs.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
That is currently the default behavior but it's a bit convoluted. If no reporter is given:
static will make one when _configure_reporter is called.
When calling report the file IO will be bypassed in favor of a concatenation as you describe.
TrajectoryReporter probably needs a rework. Both to make it compatible with h5md (https://github.com/TorchSim/torch-sim/issues/19) and to make it easier to report out model properties, as you've identified.For the latter, the simplest solution (perhaps not the best) would be to add the model_outputs as a optional kwarg to the report function so that model predictions can be covered by the current API without needing an additional call.
Originally posted by @orionarcher in https://github.com/TorchSim/torch-sim/pull/464#discussion_r2867761319
- Lenguaje dominante
- Python
- Estrellas
- 502
- Forks
- 108
- Merge medio
- 1 d 1 h
- PR fusionados (30 d)
- 12
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 TorchSim/torch-sim
-
`unwrap_positions` gives wrong results for triclinic cellsPosiblemente ocupada @lance-lii la tomó hace 3 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
TorchSim/torch-sim#629 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Autobatcher memory probe does not recognize cuequivariance out-of-memory errorsPosiblemente ocupada @lil-lon la tomó hace 4 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
TorchSim/torch-sim#627 · 5 comentarios · 1 reacción ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 2 días
-
Add MLIPOps backendsAbiertohelp wanted ml-eng-friendly
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
TorchSim/torch-sim#606 · 6 comentarios · 1 reacción ·
Los mantenedores suelen responder en 2 días
-
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
TorchSim/torch-sim#597 · 7 comentarios ·
Los mantenedores suelen responder en 2 días
Todos los issues de TorchSim/torch-sim
Issues similares
-
HandyModRTransform.deriv3 is wrong for m > 2Posiblemente ocupada @SajalDevX la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
read_messages: the first read after subscribe/psubscribe returns no messages even when some are waitingPosiblemente ocupada @matheussilvano la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 80/100
The-Swarm-Corporation/swarms-framework-docs#399 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
bump patch
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
autowarefoundation/autoware_carla_scenario#1321 ·
Los mantenedores suelen responder en 1 día