Skip to content
This repository was archived by the owner on May 21, 2020. It is now read-only.

Crear dir de nb vacíos en script#41

Merged
AlexS12 merged 3 commits intomasterfrom
Siro_Marzo_18
Mar 11, 2018
Merged

Crear dir de nb vacíos en script#41
AlexS12 merged 3 commits intomasterfrom
Siro_Marzo_18

Conversation

@AunSiro
Copy link
Copy Markdown
Member

@AunSiro AunSiro commented Mar 6, 2018

Ahora, cuando se ejecuta el script para vaciar notebooks, si el
directorio de notebooks vacíos no existe, se crea, en lugar de lanzar un
error.

Ahora, cuando se ejecuta el script para vaciar notebooks, si el
directorio de notebooks vacíos no existe, se crea, en lugar de lanzar un
error.
Copy link
Copy Markdown
Member

@AlexS12 AlexS12 left a comment

Choose a reason for hiding this comment

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

Hacía falta esto! Y si además chequease que estás en la carpeta adecuada sería genial: por ejemplo, que no estás dentro de utils o de notebooks y que te cree ahí notebooks_vacios aunque luego no encuentre ninguno. ¿Comprobamos, por ejemplo, que esté la carpeta notebooks al ejecutar el script?

El script ahora detecta si se está ejecutando en la misma carpeta donde
se descarga o en la carpeta raiz.
También respeta las celdas de código que contienen el estilo del
notebook y los gifs
@AunSiro
Copy link
Copy Markdown
Member Author

AunSiro commented Mar 9, 2018

acabo de añadir que compruebe lo de si se está ejecutando desde la carpeta y además que conserve las celdas de gifs y del estilo del notebook

Sustitucion de sumas de strings de rutas por la función os.path.join
para asegurar compatibilidad independiente de la plataforma
@AunSiro
Copy link
Copy Markdown
Member Author

AunSiro commented Mar 9, 2018

He sustituido la concatenación de strings de rutas por la función de os.path.join para asegurar que funcione independientemente del os

Copy link
Copy Markdown
Member

@AlexS12 AlexS12 left a comment

Choose a reason for hiding this comment

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

great! 🚀

Comment thread util/empty_nb.py Outdated
prepath = '../'
else: raise OSError('Carpeta de notebooks no encontrada')

vacios_path = prepath + "notebooks_vacios"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

¿podemos usar pathlib para esto? lo de concatenar strings nos va a dar problemas
https://docs.python.org/3/library/pathlib.html#basic-use

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

otra opcion con os.path es usar os.path.join

Comment thread util/empty_nb.py
source = cell['source']
if '# aeropython: preserve' in source:
continue
elif 'Image(url=' in source:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A mí esta opcion me parece buena, pero me gustaría escucha la opinion de @Juanlu001
Si nos vale podemos aprovechar a meter también los videos, que alguno hay.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Veo que ya has mergeado, pero igualmente me parece bien preservar las imágenes, los vídeos y otro material metido así en las celdas.

@AlexS12 AlexS12 merged commit 585e868 into master Mar 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants