Skip to content
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

RFC: Fixes clase 1 #17

Merged
merged 9 commits into from
Oct 6, 2020
Merged

RFC: Fixes clase 1 #17

merged 9 commits into from
Oct 6, 2020

Conversation

CrossNox
Copy link
Member

@CrossNox CrossNox commented Oct 4, 2020

  • Cambio path a datasets a gdrive
  • Agrego tags de raises-exception a las celdas que tiran excepcion
  • Agrego algunas cosas que surgieron en la clase

Cierra #21 (en teoria)

@CrossNox CrossNox changed the title WIP: Fixes clase 1 RFC: Fixes clase 1 Oct 4, 2020
@@ -34,7 +34,13 @@
# +
import pandas as pd

df = pd.read_csv('../../datasets/superheroes.csv')
GSPREADHSEET_DOWNLOAD_URL = (
Copy link
Contributor

Choose a reason for hiding this comment

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

Agregaría un comentario indicando la forma "fácil" directo de un path como para que les quede en el notebook, onda

# Para cargar un archivo csv a partir de una ruta
# df = pd.read_csv('../../datasets/superheroes.csv')

# Para cargar un archivo desde Google Sheets
<continuación>

Copy link
Member Author

Choose a reason for hiding this comment

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

Sino, en lugar de cargarlo desde gsheets podriamos descargarlo y leerlo de esta ultima forma?

Copy link
Contributor

Choose a reason for hiding this comment

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

Dale, podemos decir eso tambien

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

@@ -328,14 +390,19 @@ def alignment_to_numeric(alignment):
#
# Usamos un dataset que registra el clima y demás datos para distintas fechas de [alquiler de bicicletas](https://www.kaggle.com/c/bike-sharing-demand/data?select=train.csv)

bicis_df = pd.read_csv('../datasets/bicis.csv').set_index('datetime')
bicis_df = pd.read_csv(
Copy link
Contributor

Choose a reason for hiding this comment

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

Acá haría lo mismo que para el dataset de superheroes o aclararía en un comentario que otra forma es leer el archivo directo desde Google Sheets como para que no se asusten cuando vean esto

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed, creo

sofiamorseletto and others added 4 commits October 6, 2020 16:25
@CrossNox CrossNox merged commit 23c6d39 into master Oct 6, 2020
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.

2 participants