Skip to content

[pt-br] Localize 'sampling' page to Portuguese (Brazil)#5138

Merged
theletterf merged 38 commits into
open-telemetry:mainfrom
lopes-felipe:localization/pt-br/sampling
Sep 8, 2024
Merged

[pt-br] Localize 'sampling' page to Portuguese (Brazil)#5138
theletterf merged 38 commits into
open-telemetry:mainfrom
lopes-felipe:localization/pt-br/sampling

Conversation

@lopes-felipe

Copy link
Copy Markdown
Member

Issue: #4922

@lopes-felipe
lopes-felipe requested a review from a team September 2, 2024 23:15
@opentelemetrybot
opentelemetrybot requested review from a team September 2, 2024 23:17
@edsoncelio edsoncelio mentioned this pull request Sep 3, 2024
26 tasks

@janssenlima janssenlima left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ótimo trabalho. Deixei algumas coisas para revisão.

Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/tail-sampling-process.png
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
@opentelemetrybot
opentelemetrybot requested a review from a team September 3, 2024 13:59
lopes-felipe and others added 16 commits September 3, 2024 10:59
Co-authored-by: Janssen dos Reis Lima <janssenreislima@gmail.com>
Co-authored-by: Janssen dos Reis Lima <janssenreislima@gmail.com>
Co-authored-by: Janssen dos Reis Lima <janssenreislima@gmail.com>
Co-authored-by: Janssen dos Reis Lima <janssenreislima@gmail.com>
Co-authored-by: Janssen dos Reis Lima <janssenreislima@gmail.com>
Co-authored-by: Janssen dos Reis Lima <janssenreislima@gmail.com>
Co-authored-by: Janssen dos Reis Lima <janssenreislima@gmail.com>
Co-authored-by: Janssen dos Reis Lima <janssenreislima@gmail.com>
Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com>
Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com>
Co-authored-by: Vitor Vasconcellos <vvasconcellos1@gmail.com>
Co-authored-by: Vitor Vasconcellos <vvasconcellos1@gmail.com>
Co-authored-by: Vitor Vasconcellos <vvasconcellos1@gmail.com>
Co-authored-by: Vitor Vasconcellos <vvasconcellos1@gmail.com>
Co-authored-by: Vitor Vasconcellos <vvasconcellos1@gmail.com>
Co-authored-by: Vitor Vasconcellos <vvasconcellos1@gmail.com>
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Co-authored-by: Ezzio Moreira <enzziom@gmail.com>
Comment thread content/pt/docs/concepts/sampling/index.md Outdated

@lgfa29 lgfa29 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Muito bom, parabéns @lopes-felipe!

Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment on lines +106 to +108
## Amostragem de Cabeça

A amostragem de cabeça é uma técnica de amostragem usada para tomar uma decisão

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
## Amostragem de Cabeça
A amostragem de cabeça é uma técnica de amostragem usada para tomar uma decisão
## Amostragem pela cabeça
A amostragem pela cabeça (_head sampling_) é uma técnica de amostragem usada para tomar uma decisão

Eu não conhecia esse termo. Parece ser bastante utilizado no contexto do OpenTelemetry (a Grafana e a Honeycomb também usam) mas não encontrei nenhuma tradução.

"Pela cabeça" me parece mais próxima do sentido original, já que dá um senso de direção: a decisão de sampling é feita pelo início dos rastros. "De cabeça" parece que a cabeça está sendo amostrada 😅

Mas as duas opções me soam um pouco estranho, então não tenho muita preferência.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Boa, pensando agora acho que também prefiro "pela cabeça" ao invés de "de cabeça", mas vamos ver o que as outras pessoas tem a dizer :)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Outra opção seria um termo diferente, por exemplo "Amostragem pelo começo" e "amostragem pelo final". Isso estaria mais próximo de conceitos como começo e final de uma fila ou lista (list head/tail).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A segunda sugestão do Luís (amostragem pelo começo/pelo final) soa melhor ao invés de cabeça e calda rsrsrsrs.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fui com "pela cabeça/cauda" porque sinto que correlaciona melhor com o conceito original (head/tail) que é mais bem difundido :)
9a6192c

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

me parece fazer sentido algo como começo/final ou antecipada/tardia, mas eu acho que cabeça/cauda da pra bater o olho e saber tbm do q se trata, n tenho uma opinião forte aqui

Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
lopes-felipe and others added 5 commits September 4, 2024 16:45
Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>
Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>
Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>
Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>
Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>
Comment thread content/pt/docs/concepts/sampling/index.md Outdated
@lopes-felipe

Copy link
Copy Markdown
Member Author

Valeu pelos reviews @EzzioMoreira e @lgfa29!

@EzzioMoreira

Copy link
Copy Markdown
Contributor

Parabéns pelo trabalho 👏👏👏

@EzzioMoreira

Copy link
Copy Markdown
Contributor

LGTM!!!!!

@theletterf
theletterf enabled auto-merge September 8, 2024 14:02
@theletterf
theletterf added this pull request to the merge queue Sep 8, 2024
@opentelemetrybot
opentelemetrybot requested a review from a team September 8, 2024 16:29
Merged via the queue into open-telemetry:main with commit e71e88b Sep 8, 2024
@lopes-felipe
lopes-felipe deleted the localization/pt-br/sampling branch September 8, 2024 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

7 participants