search

pabloisaacalexz / Visualizing-math

This repository is devoted to publish scripts where I try to animate or visualize mathematical objects coming from any branch of mathematics, but specially the ones coming from probability theory.
MIT License
1 stars 0 forks source link

[BUG]: Missing argument in animateLevyConstruction #1

Closed johnny-godoy closed 1 year ago

johnny-godoy commented 1 year ago

(If you prefer for me to write in English for this repository tell me, but I'll go for Spanish now)

¡Hola Pablo!

Está bueno el repo! Estaba jugando un poco con las visualizaciones, y quería reportar un bug. La función animateLevyConstruction utiliza una variable n sin antes definirla. Esto no se nota al correrlo desde Jupyter porque defines n=11, pero si hubieras utilizado otro nombre de variable:

k = 11
t, F, B = LevyConstructionBM(k)

Entonces al correr animateLevyConstruction se cae por NameError: name 'n' is not defined. Una solución fácil de implementar es agregar n como argumento a la función, pero no sé si es la que tú prefieres.

PD: Estaba haciendo mejoras de eficiencia al código (Con cambios simples se puede hacer un boost de x28 a LevyConstructionBM). Ví que estabas abierto a aportes: ¿Prefieres que te haga un pull request para esto?

pabloisaacalexz commented 1 year ago

Hola Johnny!

Muchas gracias por tus comentarios y tus observaciones :) en virtud del tiempo, estaría encantado de que hagas un pull request para la velocidad de la función.

Decir que ambos .ipynb del repo los usé para un video de divulgación sobre el movimiento browniano y sacando las animaciones debo haber cambiado esa cosa ínfima sobre el n.

Quedo atento entonces! Saludos Pablo

El dom, 9 de abr de 2023, 15:48, Johnny Godoy @.***> escribió:

(If you prefer for me to write in English for this repository tell me, but I'll go for Spanish now)

¡Hola Pablo!

Está bueno el repo! Estaba jugando un poco con las visualizaciones, y quería reportar un bug. La función animateLevyConstruction utiliza una variable n sin antes definirla. Esto no se nota al correrlo desde Jupyter porque defines n=11, pero si hubieras utilizado otro nombre de variable:

k = 11 t, F, B = LevyConstructionBM(k)

Entonces al correr animateLevyConstruction se cae por NameError: name 'n' is not defined. Una solución fácil de implementar es agregar n como argumento a la función, pero no sé si es la que tú prefieres.

PD: Estaba haciendo mejoras de eficiencia al código (Con cambios simples se puede hacer un boost de x28 a LevyConstructionBM). Ví que estabas abierto a aportes: ¿Prefieres que te haga un pull request para esto?

— Reply to this email directly, view it on GitHub https://github.com/pabloisaacalexz/Visualizing-math/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMXY77C3ZUZSAY2ZPEATNMDXAMG7DANCNFSM6AAAAAAWYJKSLQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

johnny-godoy commented 1 year ago

Resuelto en https://github.com/pabloisaacalexz/Visualizing-math/pull/2