Skip to content

[ARTICULO] performance pandas vs numpy notebook #44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Apr 20, 2018

Conversation

asdf8601
Copy link
Contributor

@asdf8601 asdf8601 commented Apr 4, 2018

Agrego markdown e ipynb.
Ahora sí que sí. ;-)
No he podido comprobar que se renderiza todo correctamente, me decís si es necesario algún cambio.

@manugarri
Copy link
Contributor

hola @mmngreco , gracias por compartir la entrada, crees que podrias dedicar unos minutos a revisar la ortografía? Veo varias faltas que sería facil corregir (salío->salió , otar->otra, hácemos->hacemos, releevancia->relevancia, veremo->veremos , etc).

@asdf8601
Copy link
Contributor Author

asdf8601 commented Apr 18, 2018

Muchas gracias @manugarri, no me habría gustado nada que se me pasaran...

@manugarri
Copy link
Contributor

@Pybonacci/editores como lo veis? Aun hay algunas faltas ortograficas (devuelen, acentos), quizas seria una buena idea para la CI usar un corrector automatico.

@asdf8601
Copy link
Contributor Author

Usáis alguna herramienta para esto? Yo no tengo spellchecker en los notebooks... desconozco si hay alguna extensión.

@manugarri
Copy link
Contributor

de momento no @mmngreco

Copy link
Member

@astrojuanlu astrojuanlu left a comment

Choose a reason for hiding this comment

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

¡Está genial el artículo! Después de tanto tiempo que llevas esperando una revisión no quiero ni molestarte con una coma extra que he visto por ahí. Si @manugarri o @kikocorreoso están de acuerdo, mergeamos ya. ¡Gracias!

@astrojuanlu
Copy link
Member

No sé si funciona para español pero está esta extensión:

http://jupyter-contrib-nbextensions.readthedocs.io/en/latest/nbextensions/spellchecker/README.html

@asdf8601
Copy link
Contributor Author

asdf8601 commented Apr 20, 2018

Si me la ubicas no me cuesta nada corregirlo. Ya me cuesta ver las faltas @Juanlu001.
Sobre herramientas de spellchecking, estuve buscando y quizá os interese las que he encontrado https://gist.github.com/mmngreco/105934f9ccd6dc2a02f188e6c6667e07

Desconozco cuán fácil puede ser hacer funcionar spellchecker en español, he visto por ahí diccionarios español-España en .aff.

El primer enlace del gist os puede ser de utilidad.

@manugarri
Copy link
Contributor

me parece bien @Juanlu001 👍

@astrojuanlu astrojuanlu merged commit 9d8000d into Pybonacci:sources Apr 20, 2018
@astrojuanlu
Copy link
Member

Aquí está el artículo:

https://new.pybonacci.org/2018/03/27/performance-pandas-numpy-bottleneck/

Le faltaría un resumen para que se viese aquí:

https://new.pybonacci.org/

@mmngreco ¿Te animas a arreglarlo? Sería más o menos así:

https://github.com/Pybonacci/pybonacci.github.io/blame/sources/content/downloads/notebooks/ExplorandoPlanetaNueve.ipynb#L23-L32

Que no se nos olvide revisar esto la próxima vez 😅

@asdf8601
Copy link
Contributor Author

Oh, claro! @Juanlu001

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants