Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sits_services x sits_info_services #53

Closed
pedro-andrade-inpe opened this issue Feb 15, 2018 · 4 comments
Closed

sits_services x sits_info_services #53

pedro-andrade-inpe opened this issue Feb 15, 2018 · 4 comments

Comments

@pedro-andrade-inpe
Copy link
Contributor

What is the difference between sits_services and sits_info_services?

@gilbertocamara
Copy link
Contributor

Pedro, the function sits_info_services is not exported. You may be referring to sits_infoWTSS, in which case, the package documentation says:
(a) sits_services - provides information about the time series services available
(b) sits_infoWTSS - provides information about WTSS service

We envisage that SITS will provide access to different time series data provision services. Curently there are two services available: SITS from INPE and SATVEG from EMBRAPA. We plan to add other services.

It may be the case that the names of the functions are not the most appropriate. Please help us in finding better names.

@pedro-andrade-inpe
Copy link
Contributor Author

sits_info_services is not exported, but it is also not used by any other function of the package:

[pedro@~/github/sits/R] $ grep sits_info_services *
sits_info.R:#' @name sits_info_services
sits_info.R:sits_info_services <- function() {
[pedro@~/github/sits/R] $ 

Could it be removed?

@gilbertocamara
Copy link
Contributor

Point taken. I will look into the matter and come back to you.

@gilbertocamara
Copy link
Contributor

Included sits_info_services in the export list

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants