deflateBR icon indicating copy to clipboard operation
deflateBR copied to clipboard

Added support for zoo/xts time-series as input

Open dinhani opened this issue 6 years ago • 2 comments

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.

dinhani avatar Oct 02 '18 02:10 dinhani

Ó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!

meirelesff avatar Oct 02 '18 21:10 meirelesff

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.

dinhani avatar Oct 03 '18 01:10 dinhani