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

Devolución TP2 #2

Open
wants to merge 12 commits into
base: feedback2
Choose a base branch
from
Open

Devolución TP2 #2

wants to merge 12 commits into from

Conversation

eliocamp
Copy link

No description provided.

Copy link
Author

@eliocamp eliocamp left a comment

Choose a reason for hiding this comment

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

Te dejé un par de comentarios.

filter(!is.na(puntos)) |>
filter(!is.na(precio)) |>
group_by(puntos) |>
summarise(precio_promedio = mean(precio, na.rm = TRUE)) |>
Copy link
Author

Choose a reason for hiding this comment

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

En el gráfico anterior se ve que la distribución de precios tiene algunos pocos valores muy altos pero muchos con valores bajos. Quizás en vez del promedio, podés probar con la mediana.

informe.Rmd Outdated
Vamos a ver si existe algun otro rasgo relevante de la relación puntaje y precio. Primeramente creamos una nueva variable que las relacione:
```{r}
vinos_ratio <- vinos |>
mutate(ratio = precio*puntos*0.01)
Copy link
Author

Choose a reason for hiding this comment

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

No entiendo qué vendría a representar esta variable. Quizás habría que explicar un poco de donde sale y qué significa.

@RexCaecos
Copy link
Contributor

Gracias por los comentarios! Actualicé la 2da entrega, analizando la 3ra hipótesis y corrigiendo varias cosas. Luego veo de añadir este feedback.
Este es commit más actual.
bcac909

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