lbenet / 2016-2_TSFisicaComputacional

Curso de Temas Selectos de Física Computacional (2016-2)
4 stars 25 forks source link

Tarea5 #118

Closed SirSilverCharles closed 8 years ago

UriAceves commented 8 years ago

Sigues a un archivo test que no quiero y en esta te falta el archivo con las pruebas.

SirSilverCharles commented 8 years ago

@UriAceves listo, ya quite todos los archivo test de los pull requests, y agregue el archivo de las pruebas a este

UriAceves commented 8 years ago

Comenta en cada uno. Yo te comenté todos, no sólo el quinto. Así te llega un correo de cada uno que te redirige al pull en cuestión. Y no tienes que meterte a buscar cada uno. Este por ejemplo es el 118, no tengo tanto tiempo como para andar buscando en cuáles ya cambiaron cosas.

UriAceves commented 8 years ago

En el README.md dice

Como tarea 5 deben subir el notebook 08_DiferenciaciónAutomática.ipynb en su carpeta de tareas, propiamente contestado, con el mismo equipo que la tarea 4. La resolución completa de los ejercicios debe ser enviada como "Tarea5.ipynb".

en el notebook dice

NOTA

Este notebook se iniciará en la clase y debe hacerse en equipo, con los mismos miembros que participaron en la "Tarea4.ipynb". La resolución completa de los ejercicios debe ser enviada como "Tarea5.ipynb".

Yo creería que es suficientemente claro que queremos ese notebook contestado. No que tú hagas 4 celdas y ya. Leer instrucciones es una buena costumbre, podrías intentarlo.

SirSilverCharles commented 8 years ago

@UriAceves listo, puse los ejercicios en el archivo 08_DiferenciaciónAutomática.ipynb y lo subí como Tarea5.ipynb, también comente en los otros pulls que quite el archivo test.

UriAceves commented 8 years ago

Pon acentos, corrige eso de las gráficas o dame una buena razón para tener ambas, y listo.

SirSilverCharles commented 8 years ago

@UriAceves Tarea5 fix acentos, gráfica repetida

UriAceves commented 8 years ago

Si no usas UnicodePlots no lo cargues.

Talvez se podría usar el mismo método muchas veces? aunque eso introduciría un error cada vez mas y mas grande, o podríamos hacer une versión de los duales pero de segunda derivada, y luego de tercera, etc. ? o talvez se pueda hacer algo así pero automáticamente

más. Si vas a poner preguntas ábrelas "¿" esto es español. Después de "?" va una mayúscula, a menos que el signo de interrogación vaya seguido por una coma.

Fuera de eso lo demás está bien.

Las correcciones son menores, así que hago el merge.