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

Feedback #1

Open
wants to merge 6 commits into
base: feedback
Choose a base branch
from
Open

Feedback #1

wants to merge 6 commits into from

Conversation

github-classroom[bot]
Copy link
Contributor

@github-classroom github-classroom bot commented May 24, 2023

👋! 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 main since the assignment started. Your teacher can see this too.

Notes for teachers

Use this PR to leave feedback. Here are some tips:

  • Click the Files changed tab to see all of the changes pushed to main since 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”.
  • Click the Commits tab to see the commits pushed to main. Click a commit to see specific changes.
  • If you turned on autograding, then click the Checks tab to see the results.
  • This page is an overview. It shows commits, line comments, and general comments. You can leave a general comment below.
    For more information about this pull request, read “Leaving assignment feedback in GitHub”.

Subscribed: @magliweb

Copy link

@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é comentarios. Fijate que hay errores de tipeo y ortografía.

informe.Rmd Outdated Show resolved Hide resolved
informe.Rmd Outdated
Comment on lines 60 to 80
```{r}
#Poblacion_activa_uso_del_tiempo <-
uso_tiempo_2016|>
filter(edad > "18" & edad < "70" ) |>
#Poblacion_activa <-
select("trabajo_pago", "recreacion", "trabajo_domestico_no_pago", "estudio_educacion")

```



```{r}
uso_tiempo_2016 |>
select(sexo,trabajo_pago,trabajo_pago2,trabajo_domestico_no_pago,trabajo_domestico_no_pago2,tareas_cuidados, tareas_cuidados2, quintil_ingreso,rango_etario)
```

Conforme la consigna se seleccionaron las variables de interés y se asignó la info generada por la selección a una variable.

```{r}
filter(uso_tiempo_2016, edad == "14")
```

Choose a reason for hiding this comment

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

Acá también el código devuelve tablas enteras y no guarda la selección ni el filtro. Recordá que la tabla nunca se modifica, sino que las acciones de filtrar, seleccionar, etc... devuelven un nueva tabla con el resultado de esas acciones. Para guardar ese resultado hay que asignarlo a una variable.

informe.Rmd Outdated
Comment on lines 83 to 86
```{r}
uso_tiempo_2016|>
count("Varon", sexo)

Choose a reason for hiding this comment

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

Esto devuelve lo mismo que está más arriba salvo que agrega una columna con el valor constante "Varon". No hace falta poner ese "Varon".

informe.Rmd Outdated
cuenta la cantidad de varones y mujeres encuestadas

```{r}
summarize(uso_tiempo_2016, proporción_media = mean ("estudio_educacion", na.rm = TRUE))

Choose a reason for hiding this comment

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

Esto te está dando NA porque estás calculando la media del texto "estudio_educacion" que, como es un texto, no tiene media. Tenés que sacarle las comillas.

informe.Rmd Outdated
uso_tiempo_2016|>
filter(edad >"18") |>
group_by(sexo) |>
summarize( tiempo_Max = max(trabajo_pago), tiempo_Mim = min(trabajo_pago), desvio = sd(trabajo_pago), na.rm = TRUE)

Choose a reason for hiding this comment

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

El na.rm = TRUE tiene que ir dentro de las funciones max, sd, etc. Acá está como un argumento a summarize() y lo que hace es agregar una columna con el nombre na.rm y valor TRUE. En estos casos no hay valores faltantes, así que no hace falta que lo pongas.

informe.Rmd Outdated
Comment on lines 107 to 111
```{r}
uso_tiempo_2016|>
filter(edad > "18" & edad < "70", ) |>
select( "trabajo_pago", "recreacion", "trabajo_domestico_no_pago", "tareas_cuidados")

Choose a reason for hiding this comment

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

Esto también devuelve un montón de datos.

informe.Rmd Outdated
select( "edad","trabajo_pago", "trabajo_domestico_no_pago", "tareas_cuidados")
```

Agrupamos por sexo, para seleccionar solo los varones y luego filtramos por edad lavorable

Choose a reason for hiding this comment

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

En el código la agrupación está comentada así que no corre. El grupo no selecciona únicamente a los varones. Quizás este texto te quedó de una versión anterior del código?

informe.Rmd Outdated
Comment on lines 148 to 154
```{r}
ggplot()+
geom_line(data=vargrap,aes(y=mediadelhogar,x= edad,colour="mediadelhogar"),size=1 )+
geom_line(data=vargrap,aes(y=trabajopago,x= edad,colour="trabajopago"),size=1) +
scale_color_manual(name = "Promedios", values = c("mediadelhogar" = "darkblue", "trabajopago" = "red")) +
labs(title='Tiempo ocupado en tareas hogareneas VS trabajo pago', y = "Tiempo empleado")

Choose a reason for hiding this comment

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

Para hacer este gráfico lo que te conviene es usar pivot_longer() para alargar la tabla y luego usar un solo geom_line(). Algo así:

vargrap |> 
  pivot_longer(cols = -edad) |>    # alarga todas las columnas menos edad
  ggplot(aes(edad, value)) +
  geom_line(aes(color = name))

informe.Rmd Outdated
Comment on lines 173 to 193
```{r}
vargrap2 <- rbind (
vargrap2 |> filter(edad < 23) |>
summarise(edad = "grupo 1",prom =mean(Promedio_Aplicaciones), dev = mean(DesvStandard)),

vargrap2 |> filter(edad >= 23 & edad < 35 ) |>
summarise(edad = "grupo 2",prom =mean(Promedio_Aplicaciones), dev = mean(DesvStandard)),

vargrap2 |> filter(edad >= 35 & edad < 45 ) |>
summarise(edad = "grupo 3",prom =mean(Promedio_Aplicaciones), dev = mean(DesvStandard)),

vargrap2 |> filter(edad >= 45 & edad < 55 ) |>
summarise(edad = "grupo 4",prom =mean(Promedio_Aplicaciones), dev = mean(DesvStandard)),

vargrap2 |> filter(edad >= 55 & edad < 70 ) |>
summarise(edad = "grupo 5",prom =mean(Promedio_Aplicaciones), dev = mean(DesvStandard))
)



```

Choose a reason for hiding this comment

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

Ojo que en los datos ya hay una variable llamada rango_etario que tiene la edad categorizada por rango. Si querés hacer un recorte con otras edades, podés usar la función cut() para crear una nueva variable edad_categorica (o como quieras llamarla) con la edad por rangos y luego agrupar por esa variable.

informe.Rmd Outdated

```{r}
vargrap3 <- uso_tiempo_2016 |>
filter(edad >= 14 & edad < 25 ) |>

Choose a reason for hiding this comment

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

Acá podrías no filtrar por edad para ver toda la curva.

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.

1 participant