search

ipeadata-lab / ipeaplot

ipeaplot: criando graficos no padrão editorial do Ipea
https://ipeadata-lab.github.io/ipeaplot/
Other
2 stars 0 forks source link

Parameter `pt_br` is not obvious enough #15

Closed rafapereirabr closed 1 year ago

rafapereirabr commented 1 year ago

The name os parameters should be self-evident. The parameter pt_br is not. I would suggest we use the following:

#' @param decimal A string indicating whether the decimal separator should be a
#'        `","` or a `"."`. By default, the function uses a comma `","`, following
#'        the format used in Brazilian Portuguese.
PedroJorge7 commented 1 year ago

Eu concordo. Vou fazer a modificação no pacote.

PedroJorge7 commented 1 year ago

Segui a sugestão do Pedro na nossa call e fui olhar como o kable ajusta a questão dos decimais. Me parece que eles fazem a alteração usando os argumentos decimal.mark = ","e big.mark = ".". Utilizamos esses parâmetros também, mas dentro do pt_br = T. Vou pensar um pouco se consigo deixar mais claro esse parâmetro. Ao menos que vocês achem melhor separar o decimal.mark e big.mark

rafapereirabr commented 1 year ago

decimal.mark é bom

PedroJorge7 commented 1 year ago

Então vou fazer alteração colocando decimal.mark e big.mark como parâmetros

rafapereirabr commented 1 year ago

acho que pode colocar um parametro só. "decimal.mark"

Se o "decimal.mark" é virgula, o marcador de milhares automaticamente vira ponto, e vice-versa

PedroJorge7 commented 1 year ago

Inclui o decimal.mark dentro dessas condições:

Se o "decimal.mark" é virgula, o marcador de milhares automaticamente vira ponto, e vice-versa