-
Notifications
You must be signed in to change notification settings - Fork 70
Dynamic equations #18
Conversation
f9f90cc to
26b3564
Compare
|
Declaro inaugurado el proceso de revisión de esta rama. |
|
Estoy pendiente de dos cosas para terminar con este pull request:
|
|
Nos esperamos entonces a que esté terminado para revisar. |
Dynamic equations
one test was not passing!
|
Ok! Esta rama está lista a falta del example_001.py que es una prueba muy burda. ¿Lo dejo o lo quito para hacer el merge? Se lo dejo a @Juanlu001 para que lo revise. @DLpadilla, mira los últmos commits: https://github.com/AeroPython/PyFME/pull/18/commits Porque he corregido un par de cosas de estilo y sobre todo de cómo se hace el archivo de tests. |
examples/example_001.py
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
De aquí para abajo lo metería en un bloque if __name__ == '__main__', para que se diferencie claramente.
|
He hecho un pequeño comentario en el ejemplo, pero en realidad creo que molaría más pasarlo a un notebook 😄 |
Todavía no está terminada, pero voy lanzando el pull request para que conste en el repositorio oficial que esto avanza.
He implementado el integrador en una clase porque me parecía la forma en que se podía usar de manera más sencilla desde el example_001.py (para quien quiera ir jugando). Ya hablaremos de clases, de momento que nadie se preocupe y ya discutiremos como implementar esto para que podamos elegir el sistema de ecuaciones con facilidad.
Creo que hay algún signo mal en la ecuación de fuerzas (una fuerza positiva según el eje z cuerpo debería hacer que se cayese y ahora mismo sube) Hay que revisarlo.(Fz > 0, sin rotaciones, en ejes cuerpo hace que baje z>0 en ejes horizonte)Los jacobianos parece que están bien, pero convendría echarles otro vistazo e implementar el de las ecuaciones de navegación. Todo esto cantará con los tests, que ya están en marcha.