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

Tradução da base tidyr::household #95

Merged
merged 5 commits into from
Nov 14, 2023
Merged

Tradução da base tidyr::household #95

merged 5 commits into from
Nov 14, 2023

Conversation

beatrizmilz
Copy link
Collaborator

No description provided.

@beatrizmilz
Copy link
Collaborator Author

@halian-vilela, essa base é bem curtinha, caso queira revisar!

Copy link
Contributor

@scopinho scopinho left a comment

Choose a reason for hiding this comment

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

The translation looks good to me. Using on R4DS 2e (Portuguese translation).

inst/specs/household.yml Outdated Show resolved Hide resolved
inst/specs/household.yml Outdated Show resolved Hide resolved
@@ -0,0 +1,25 @@
df:
Copy link
Contributor

@scopinho scopinho Nov 13, 2023

Choose a reason for hiding this comment

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

Peguei este erro posterirmente ao testar o df traduzido no livro R4DS. Este tibble é usado no R4DS (2e) para demonstrar o uso do .value na pivot_longer. Se usarmos data_nascimanto este () impacta o código no livro com resultado incorreto. Por isso, sugiro usarmos (.) ou outra forma sem adicionar um underline () extra no nome da coluna.

Codigo:
nucleo_familiar |> pivot_longer(cols = !familia, names_to = c(".value", "crianca"), names_sep = "_", values_drop_na = TRUE)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Bem observado!
Para ficar mais próximo ao livro, tem duas opções:

  • datanascimento_crianca1 sem separar o data nascimento
  • dn_crianca1 dn seria o equivalente ao dob (acho que esse fica mais parecido!!)
    O que acha?

Copy link
Contributor

Choose a reason for hiding this comment

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

dn_crianca fica mais proximo, porém diferente do "dob" em US, "dn" no BR não é muito utilizado. Sugiro ficar com "datanascimento_crianca1", pois apesar de não impactar neste exemplo específico, fica mais intuitivo caso a base for usado em outros lugares ou exemplos em treinamentos, etc. O que acha?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Faz sentido, dob é uma sigla mais usada! Pode ser então. Vou alterar aqui! Muito obrigada!!

@rivaquiroga rivaquiroga merged commit b8b89f9 into master Nov 14, 2023
3 checks passed
@rivaquiroga rivaquiroga deleted the tidyr-household branch November 14, 2023 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

4 participants