Feedback #1
Feedback #1
Conversation
![github-classroom[bot]](https://avatars.githubusercontent.com/in/68187?s=60&v=4){kind=small}
There was a problem hiding this comment.
Ahí te hice comentarios en el código.
Además de los comentarios, en términos generales, falta mucho texto. Describir los datos: ¿qué columnas tiene? ¿Cuántos datos? Y sobre los resultados, tablas y gráficos, ¿qué motiva lo que mostrás y qué conclusiones sacás?
informe.Rmd
Outdated
| ```{r} | ||
| library(dplyr) | ||
| ``` |
There was a problem hiding this comment.
No hace falta que cada library() esté en un bloque separado. Poné todos en el primero (el que dice setup) arriba de este.
informe.Rmd
Outdated
| ``` | ||
|
|
||
|
|
||
| ## Introducción | ||
| ```{r} | ||
| library(gitcreds) |
There was a problem hiding this comment.
Este paquete no hace falta cargarlo. No usás ninguna función en el código .
informe.Rmd
Outdated
|
|
||
|
|
||
| ```{r} | ||
| library(usethis) |
There was a problem hiding this comment.
Este paquete tampoco hace falta cargarlo
informe.Rmd
Outdated
|
|
||
|
|
||
|
|
||
| ## Exploración de los datos | ||
|
|
||
| ```{r} | ||
| library(readr) |
There was a problem hiding this comment.
El paquete ya lo cargaste arriba, sólo hace falta cargarlo una sola vez.
informe.Rmd
Outdated
|
|
||
|
|
||
|
|
||
| ## Exploración de los datos | ||
|
|
||
| ```{r} | ||
| library(readr) | ||
| captura_puerto_flota_2010_2018 <- read_csv("datos/captura-puerto-flota-2010-2018.csv") |
There was a problem hiding this comment.
Estos datos no están en el repositorio. Tienen que estar para que pueda correr esto.
There was a problem hiding this comment.
Acá también hay un posible probelma de codificación (habrás visto que las tildes se ven mal).
Esto es porque el archivo está con una codificación no estándar. Tenés que definir la codification así:
captura_puerto_flota_2010_2018 <- read_csv("datos/captura-puerto-flota-2010-2018.csv",
locale = locale(encoding = "windows-1252"))
There was a problem hiding this comment.
Gracias! me la pase investigando y no lo pude solucionar
informe.Rmd
Outdated
| captura_puerto_flota_2010_2018 |> | ||
| filter ( fecha %in% c("2010-04", "2010-05", "2010-06")) |> | ||
| group_by(puerto) |
There was a problem hiding this comment.
Igual que en bloques anteriores, esto te devuelve el resultado del filtro pero no lo guarda en ningún lado y no es útil para verlo en un informe.
informe.Rmd
Outdated
| aparece_especie <- captura_puerto_flota_2010_2018 %>% | ||
|
|
||
| filter(fecha=="2010-01") |> | ||
| group_by(especie) |> | ||
| tally() |
There was a problem hiding this comment.
Este resultado no lo estás mostrando en el informe.
Quizás contar por especie que queda demasiadas especies. Vi que hay una columna llamada especie_agrupada que tiene menos categorías; quizás podés usar esa para analizar .
informe.Rmd
Outdated
| ahora le asigno una variable a cada cosa y saco la media | ||
|
|
||
| ```{r} | ||
| captura_2010_01_Merluzahubbsi/captura_2010_01_todas |
There was a problem hiding this comment.
Esto no es bueno hacerlo porque en general no tiene sentido dividir una tabla por otra. En este caso en particular funciona porque es un caso muy particular. Está bien lo que hiciste más arriba:
mutate(proporcion = captura[especie = "Merluza hubbsi"]/captura)
informe.Rmd
Outdated
| especies_repre |> | ||
| ggplot(data = especies_repre, mapping = aes(x = fecha, y = captura)) + | ||
| geom_line (aes(colour = especie)) |
There was a problem hiding this comment.
Este gráfico te sale mal porque la fecha es una columna de tipo texto. Si la pasás a fecha va a salir bien.
informe.Rmd
Outdated
| especies_repre |> | ||
| pivot_wider(names_from = especie, values_from = captura, values_fill = 0) |
There was a problem hiding this comment.
Esto está bien el código, pero me parece que el resultado no es muy útil porque, por lo que veo en los datos originales cada fila representa la captura de una especie. Entonces pivot_wider() te deja ceros en todas las especies salvo una en cada fila.
👋! GitHub Classroom created this pull request as a place for your teacher to leave feedback on your work. It will update automatically. Don’t close or merge this pull request, unless you’re instructed to do so by your teacher.
In this pull request, your teacher can leave comments and feedback on your code. Click the Subscribe button to be notified if that happens.
Click the Files changed or Commits tab to see all of the changes pushed to
mainsince the assignment started. Your teacher can see this too.Notes for teachers
Use this PR to leave feedback. Here are some tips:
mainsince the assignment started. To leave comments on specific lines of code, put your cursor over a line of code and click the blue + (plus sign). To learn more about comments, read “Commenting on a pull request”.main. Click a commit to see specific changes.For more information about this pull request, read “Leaving assignment feedback in GitHub”.
Subscribed: @leonelcejas774