Skip to content

Conversation

@santisoler
Copy link
Member

Make small changes for the Geolatinas Coding Group course.
Add formative assesments and some more exercises and discussions.

Make small changes for the Geolatinas Coding Group course.
Add formative assesments and some more exercises and discussions.
@santisoler
Copy link
Member Author

Empecé a editar un poco este notebook. Creo que le faltaban varios formative assesment, y me copó bastante la idea de armar los breakup rooms para que resuelvan ejercicios, así que agregué uno más al final.

Diganme qué les parece, podemos cambiar lo que quieran.

@santisoler
Copy link
Member Author

Ahh, una cosa! Podriamos usar ReviewNB para hacer reviews de los cambios en los notebooks.
Habría que activarlo para este repositorio, pero como no soy admin de este grupo creo que no puedo hacerlo.
Hay una opción gratuita, así que no hay q preocuparse por pagar.

@andieie
Copy link
Contributor

andieie commented Feb 3, 2021

Ahh, una cosa! Podriamos usar ReviewNB para hacer reviews de los cambios en los notebooks.
Habría que activarlo para este repositorio, pero como no soy admin de este grupo creo que no puedo hacerlo.
Hay una opción gratuita, así que no hay q preocuparse por pagar.

Ya lo instale y puedo ver los cambios 💯 . Tu puedes acceder ?

@santisoler
Copy link
Member Author

Sisí, están acá: https://app.reviewnb.com/GeoLatinas/Intro-to-python/pull/2/

Ahora que lo tenemos instalado, cuando creemos un nuevo PR debería aparecer un comentario de un bot con el enlace a ReviewNB para el mismo PR. No se si hay que configurar algo más o si lo hace por defecto. Probemos más adelante 🤞🏼

"Usar **variables** nos permite _guardar_ valores bajo determinados nombres.\n",
"Por ejemplo, asignemos una cantidad hipotética de alumnes a una variable y el nombre del curso en otra."
]
},
Copy link
Contributor

Choose a reason for hiding this comment

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

oki aqui es que iniciamos los breakout rooms correcto ?


Reply via ReviewNB

Copy link
Member Author

Choose a reason for hiding this comment

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

Sii! Estos ejercicios los dejaríamos para que los resuelvan elles soles.

Capaz que podemos armar un poll para que cada ver qué tal les fue, si obtuvieron los resultados correctos, aunque por ahí estaría bueno que alguien muestre cómo lo hizo y damos feedback de manera general.

Copy link
Contributor

Choose a reason for hiding this comment

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

Creo que es mejor que muestren lo que hicieron. No queremos marearlos con polls despues de los breakout rooms que ya es un mareo.

Copy link
Member Author

Choose a reason for hiding this comment

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

Sii tenés razón!

@MGomezN
Copy link

MGomezN commented Feb 7, 2021

En la minuta del 13 de enero (Logística) dice
File system:
README.md
notebooks/
datos/

Entonces hace falta meter los csv, txt y dat a una carpeta de datos? Lo hago?

@santisoler
Copy link
Member Author

En la minuta del 13 de enero (Logística) dice
File system:
README.md
notebooks/
datos/

Entonces hace falta meter los csv, txt y dat a una carpeta de datos? Lo hago?

Sí, estaría bueno hacer eso! Incluso podemos usar la oportunidad para mostrar cómo abrir archivos que no están en el mismo directorio que el notebook.
Aunque yo diría de abrir un nuevo PR para eso.

Dado que @andieie aprovó este PR, lo voy a mergear. Cualquier cosa que quieran cambiar, siéntanse libres de abrir otro PR.

@santisoler santisoler merged commit b8ebcb0 into main Feb 9, 2021
@santisoler santisoler deleted the santi-edit-variables branch February 9, 2021 13:05
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.

4 participants