-
Notifications
You must be signed in to change notification settings - Fork 71
Conversation
|
What do you think about the check? as far as I know, np.isclose is not implemented for numpy 1.6... should we take it into consideration? |
|
By no means. The 1.6.x series of NumPy is incredibly buggy and 1.6.2 was released almost four years ago. With modern packaging tools it makes no sense to support such an old version, so it's safe to say we strive for NumPy 1.8 or even NumPy 1.9. |
|
Copy that! |
|
I hear your concerns. Paraphrasing Nick Coghlan, "Stop Supporting NumPy 1.6 (For Free)" 😉 http://www.curiousefficiency.org/posts/2015/04/stop-supporting-python26.html |
|
Quote from the article:
The point here is that there might be good reasons for big companies to have ancient technology stacks, but that there are no reasons for us to support those for free in our volunteer time. So yeah, if [random aeronautical multinational company] wants to use this software with Python 2.6 and NumPy 1.6 on Windows XP, they can hire us 😄 |
|
que opinais de la nomenclatura? obviamente euler_angles, quaternion(que tiene sus cuatro componentes dentro: q_0, q_1, q_2, q_3), quaternion_vel que es la derivada del cuaternion y los elementos de quaternion_vel sean d_q0, dq_1, dq_2, dq_3 |
|
Esta semana buscamos un día para mirar esto juntos, vale? |
|
Sería bastante interesante hacer merge de esta rama, ya que en #65 tengo contemplado que el sistema mantenga los quaterniones actualizados aunque trabaje con ángulos de euler para integrar. La verdad es que no recuerdo muy bien el estado de esto. @JuanMatSa nos puedes comentar algo? estaba para revisar ya? |
|
@JuanMatSa dinos en qué estado está esto y @AlexS12 si se puede postponer para 0.2 lo dejamos pendiente. |
|
@JuanMatSa told me that he is no longer participating in the project, at least for the next months. It is my intention to merge this, but I would like someone to review the code and I have my own comments. hands up ✋?? @AeroPython/pyfme @Juanlu001, what is the best approach for a "dead" pull request? should I clone the branch, and start a new pull request and close this one? |
|
Yes, do exactly that! Assuming you cannot already push to his branch. |
|
Maybe we can incorporate some ideas from this repo: https://github.com/satellogic/quaternions |
AlexS12
left a comment
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.
We are going to merge, and changes will come later in case it is necessary
|
|
||
| q_0, q_1, q_2, q_3 = quaternion | ||
|
|
||
| check_unitnorm(quaternion) |
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.
Maybe this is too much if it has to be evaluated each time step. But I think we'll have time to get rid of it later
Still a lot of things to implement, like tests, docstrings... at least the equations to transform quaternions to euler angles (and euler angles to quaternions) are implemented.