search

tchiluanda / rtn

Other
3 stars 1 forks source link

Import/Depend issue #3

Open Figuera opened 2 years ago

Figuera commented 2 years ago

A arquivo de descrição do pacote lista todas as dependências como Depends:

Package: rtn
Type: Package
Title: Data manipulation from Relaório do Tesouro Nacional (RTN) database
Version: 0.1.0
Author: Fernando Almeida Barbalho
Maintainer: Fernando Almeida Barbaçho <fbarbalho@gmail.com>
Description: Retrieves data from RTN database.
License: MIT + file LICENSE
Encoding: UTF-8
LazyData: true
Depends: 
    R (>= 2.10),
    ckanr,
    tidyverse,
    lubridate,
    tidyr,
    zoo,
    forecast,
    readxl,
    dplyr
Imports: 
    magrittr
RoxygenNote: 7.1.1
Suggests: 
    testthat (>= 3.0.0)
Config/testthat/edition: 3

O que força o usuário do pacote a carregar todos esses pacotes em sua sessão quando utiliza o pacote, o que não é necessário e aumenta o risco que algum desses pacotes sobrescreva alguma função já definida pelo usuário. Além disso alguns desses pacotes são redundantes, dplyr e tidyr já fazem parte de tidyverse.

Me parece melhor que essas dependências sejam importadas, algo como:

Package: rtn
Type: Package
Title: Data manipulation from Relaório do Tesouro Nacional (RTN) database
Version: 0.1.1
Author: Fernando Almeida Barbalho
Maintainer: Fernando Almeida Barbaçho <fbarbalho@gmail.com>
Description: Retrieves data from RTN database.
License: MIT + file LICENSE
Encoding: UTF-8
Depends:
    R (>= 2.10)
Imports: 
    ckanr,
    lubridate,
    zoo,
    forecast,
    readxl,
    tibble,
    dplyr,
    stringr,
    tidyr,
    magrittr
RoxygenNote: 7.1.1
Suggests: 
    testthat (>= 3.0.0)
Config/testthat/edition: 3
fernandobarbalho commented 2 years ago

Bem colocado. Eu tive problemas na importação. Quick and dirty, tive que apelar para o depends. Agora que tenho mais tempo vou investigar melhor os problemas que ocorreram com a importação.

Figuera commented 2 years ago

Eu testei a minha proposta e tudo pareceu funcionar corretamente. Eu também retirei LazyData = True porque me parece desnecessário.

Não foram testes extensivo mas acredito que deve funcionar.

fernandobarbalho commented 2 years ago

Legal. Vou testar aqui então. Quando eu usei o import estava apresentando alguns problemas de métodos não reconhecidos. Não quis parar muito tempo para ver o que era por conta do prazo como o depends estava funcionondo deixei isso para outro momento. Agora vou ver com bem calma e acerto esses ponteiros.