Skip to content
This repository has been archived by the owner on Aug 13, 2020. It is now read-only.

Consistencia en las cabeceras de las funciones - pasar arrays o números #21

Closed
AlexS12 opened this issue Jan 5, 2016 · 7 comments
Closed
Assignees
Milestone

Comments

@AlexS12
Copy link
Member

AlexS12 commented Jan 5, 2016

De momento no pasa nada (lo pongo para aquí para acordarnos), pero para la siguiente vez que nos juntemos deberíamos llegar a un acuerdo sobre como pasar los argumentos que son vectores, si como arrays o por cada una de sus componentes. De momento tenemos ejemplos de ambas:

linear_and_angular_momentum_eqs(time, vel, mass, inertia, forces, moments)

body2hor(body_coords, theta, phi, psi)

otra opción más avanzada, como comentaba @Juanlu001 en el grupo de Telegram, sería hacer algo parecido a lo que hace SymPy o usarlo directamente.

@AlexS12
Copy link
Member Author

AlexS12 commented Jan 13, 2016

El consenso es que siempre que sea posible pasaremos los vectores, en lugar de las componentes. ls nombres recomendados son:

lin_vel = (u, v, w)
ang_vel = (p, q, r)
pos = (x, y, z)
attitude = (theta, phi, psi)

alpha y beta se pasarán separados

@AlexS12 AlexS12 closed this as completed Jan 13, 2016
@astrojuanlu
Copy link
Member

¡Esto merece una entrada en la wiki también!

@AlexS12
Copy link
Member Author

AlexS12 commented Jan 14, 2016

Ok! Me lo apuntó. Reabro el issue para acordarme, igual requiere cambiar cosas en código. Echaré un vistazo también!

@AlexS12 AlexS12 reopened this Jan 14, 2016
@astrojuanlu
Copy link
Member

Esto merece una entrada en la wiki @AlexS12 😉

@astrojuanlu astrojuanlu added this to the v0.1 milestone Jul 7, 2016
@AlexS12
Copy link
Member Author

AlexS12 commented Jul 9, 2016

Esto afectaría fundamentalmente a las funciones que hicimos al principio del simulados. Ahora, la verdad es que usando poo y pasando las clases que os enseñé por cabecera, se simplifica bastante el tema.

Me apunto la entrada en la wiki! 👍

@astrojuanlu
Copy link
Member

Supongo que esto está relacionado con #41, ¿qué opináis?

@AlexS12
Copy link
Member Author

AlexS12 commented Aug 17, 2016

Sí, yo creo que vamos a cerrar este y dejamos el otro abierto para no olvidarnos

@AlexS12 AlexS12 closed this as completed Aug 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants