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

[WIP] Input generator #56

Closed
wants to merge 12 commits into from
Closed

Conversation

olrosales
Copy link
Member

No description provided.

Copyright (c) AeroPython Development Team.
Distributed under the terms of the MIT License.

<Inputs generator>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Los <> se pueden quitar 😉

@AlexS12
Copy link
Member

AlexS12 commented Feb 16, 2016

Buena contribución, además me parece elegante la forma de hacerlo. Ya te he torturado con unos cuantos comentarios ya me dirás qué te parecen!

@olrosales
Copy link
Member Author

No he hecho los test por si alguien quería hacerlos, de no ser así los hago.

@AlexS12
Copy link
Member

AlexS12 commented Feb 23, 2016

Para mí imposible esta al menos hasta el domingo. Si no está para entonces me pondré con esto. Si no meteré mano a otra cosa 😉

@olrosales
Copy link
Member Author

what is the problem? It is related to python 2.7

@AlexS12
Copy link
Member

AlexS12 commented Apr 11, 2016

 T = 4
 A = 3
 time = np.linspace(0, 10, 11)
expected_input = np.zeros([11])
expected_input[0:5] = np.array([0, A/4, A/2, 3*A/4, A])

I has to do with integer division A/2, A/4 ....

Here you are some info about it!

http://stackoverflow.com/questions/183853/in-python-what-is-the-difference-between-and-when-used-for-division?answertab=votes#tab-top

@olrosales
Copy link
Member Author

@AlexS12 I choosed to turn A=2 into A=2. but if you think it is better to use from __future__ import division , I can change it.

@AlexS12
Copy link
Member

AlexS12 commented Apr 20, 2016

Hola @olrosales veo que esto progresó bastante la semana pasada, pero que sigue poniendo WIP. Está ya terminado? Si es así le echo una última ojeada (ya he estado viendo y pinta muy bien) y hacemos el merge. Tú me dices!

@AlexS12
Copy link
Member

AlexS12 commented Aug 16, 2016

¿Cómo va esta parte? estamos en un momento en el que podría ser bastante útil incorporarla. Cuando tengas un rato cuéntanos, @olrosales

@olrosales
Copy link
Member Author

@AlexS12 yo creo que eso lo subi como por febrero

@olrosales
Copy link
Member Author

@olrosales Octubre perdon

@astrojuanlu
Copy link
Member

Este código pasa todas las pruebas, está bien documentado y tiene pinta de estar completo. Lo he revisado y lo encuentro aceptable, si @olrosales confirmas que está terminado y @AlexS12 das el visto bueno lo metemos y cerramos esto.

@AlexS12
Copy link
Member

AlexS12 commented Aug 22, 2016

cerramos para mergear en #73

@AlexS12 AlexS12 closed this Aug 22, 2016
@olrosales olrosales deleted the input_generator branch November 13, 2016 07:34
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.

None yet

3 participants