deflateBR
deflateBR copied to clipboard
Added support for zoo/xts time-series as input
A maioria dos pacotes que trabalham com dados financeiros geralmente aceitam séries temporais dos pacotes zoo
e xts
como entrada. Adicionei uma checagem adicional para poder permitir trabalhar com esse objetos.
Se achar que isso é válido, precisa atualizar a documentação e pensar se a função deflate
deve retornar um objeto desse tipo (zoo
ou xts
) quando a entrada for desse tipo.
Ótima sugestão, @dinhani. Seria realmente legal expandir as possibilidades de input e output. Mas tenho que ver como implementar isso. No travis, teu pull mesmo não compila. Importar os pacotes seria o ideal, mas geralmente sou contra incluir muitas dependências. Vou dar uma estudada nesse formato, talvez exista uma alternativa.
De qualquer forma, vou deixar esse pull aberto -- até para eu mesmo lembrar disso. Abs!
Sobre o Travis, ainda não tenho muita familiaridade com a construção de pacotes no R, parece que foi mais por conta de não ter declarado o uso da função installed.packages
no NAMESPACE.
Undefined global functions or variables: installed.packages
Consider adding importFrom("utils", "installed.packages") to your NAMESPACE file.
Também gerou um warning sobre usar o pacote zoo
sem declarar lá, mas nesse caso foi intencional.
'::' or ':::' import not declared from: ‘zoo’checking R code for possible problems ... NOTE
Concordo com evitar que o pacote com muitas dependências. Por isso mesmo eu deixei a checagem dinâmica, ela só ocorre caso o usuário já tenha o pacote zoo
instalado, mas não força nada caso não tenha.
if ("zoo" %in% rownames(installed.packages()) && zoo::is.zoo(nominal_values)) {`
Bom, fica a ideia.