Skip to content

Conversation

@dinhani
Copy link

@dinhani dinhani commented Oct 2, 2018

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.

@meirelesff meirelesff added the enhancement New feature or request label Oct 2, 2018
@meirelesff
Copy link
Owner

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

@dinhani
Copy link
Author

dinhani commented Oct 3, 2018

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants