lbenet / 2016-2_TSFisicaComputacional

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

Tarea6 #134

Closed luyves closed 8 years ago

luyves commented 8 years ago

@artcidp , @LeoGalPras y mía. Faltan los tests.

luyves commented 8 years ago

En el último commit me refiero a que quité el archivo que Arturo accidentalmente añadió (a la carpeta que no era), no que lo borré yo sin querer. Sólo aclaro.

UriAceves commented 8 years ago

Puntos menos por ser ambiguo.

a = Taylor([0,0,1])
b = Taylor([1,1,1,1])
b/a

me da

Taylor{Float64}([Inf,Inf,NaN,NaN,NaN,NaN])

Es el único problema que he encontrado. Cuando tengan sus test comenten y vuelvo a revisar.

luyves commented 8 years ago

El método tiene problemas al tratar de hacer la serie de Taylor de algo que diverge en x = 0.

Ya corregí el código y, si tomamos a = Taylor([1,0,1,0,0]) b = Taylor([1,1,1,1]) b/a

nos da

Taylor([1,1,0,0,0,0,1,1]) que es consistente con lo que nos dice Wolfram. Tomé a con más términos 0 de orden superior pues el método lo que hace es truncar la aproximación al rango de a.

Aún faltan los tests, cuando los acabemos comento de nuevo.

UriAceves commented 8 years ago

Sólo un chequeo rápido, ¿qué les da si hacen esto?

a = Taylor([0,0,1,0,0])
b = Taylor([0,0,0,0,1])
b/a
luyves commented 8 years ago

Tienes razón, no funcionaba. No había vuelta de hoja así que borré la definición que teníamos de la división 1/g e implementé la correcta para f/g (re-utilicé nuestro código así que no fue gran cosa).

Debería de funcionar bien ahora.

luyves commented 8 years ago

Ya están los tests.

UriAceves commented 8 years ago

Bien, buena tarea.

Hago el merge.