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

Conversation

@JuanMatSa
Copy link
Member

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.

@JuanMatSa
Copy link
Member Author

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?

@astrojuanlu
Copy link
Member

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.

@JuanMatSa
Copy link
Member Author

Copy that!
Some companies still use 1.6 ¬¬

@astrojuanlu
Copy link
Member

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

@astrojuanlu
Copy link
Member

Quote from the article:

if you're currently supporting Python 2.6 for free because folks using RHEL 6 or one of its derivatives say they need it, and this is proving to be a hassle for you, then stop. If they complain, then point them at this post, as providing an easily linkable reference for that purpose is one of the main reasons I'm writing it. If they still don't like it, then you may want to further suggest that they come argue with me about it, and leave you alone.

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 😄

@JuanMatSa
Copy link
Member Author

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

@AlexS12
Copy link
Member

AlexS12 commented Jan 25, 2016

Esta semana buscamos un día para mirar esto juntos, vale?

@AlexS12
Copy link
Member

AlexS12 commented Jun 21, 2016

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?

@astrojuanlu
Copy link
Member

@JuanMatSa dinos en qué estado está esto y @AlexS12 si se puede postponer para 0.2 lo dejamos pendiente.

@AlexS12
Copy link
Member

AlexS12 commented Nov 29, 2017

@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?

@astrojuanlu
Copy link
Member

Yes, do exactly that! Assuming you cannot already push to his branch.

@AlexS12
Copy link
Member

AlexS12 commented Nov 29, 2017

Maybe we can incorporate some ideas from this repo: https://github.com/satellogic/quaternions

Copy link
Member

@AlexS12 AlexS12 left a 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)
Copy link
Member

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

@AlexS12 AlexS12 merged commit c1e1817 into AeroPython:master Dec 10, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants