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

Improved handling return_fields #15

Open
stijnvanhoey opened this issue Nov 29, 2017 · 3 comments
Open

Improved handling return_fields #15

stijnvanhoey opened this issue Nov 29, 2017 · 3 comments

Comments

@stijnvanhoey
Copy link
Collaborator

Currently, the return-fields for each of the calls are hard-coded in the package

in get_variables:

    return_fields <- c("station_name", "station_no", "ts_id",
                       "ts_name", "parametertype_name",
                       "stationparameter_name")

in get_timeseries_tsid:

    # define the date fields we require
    return_fields <- c("Timestamp", "Value", "Quality Code")

and in get_stations:

    custom_attributes <- c("dataprovider")
    return_fields <- c("custom_attributes", "station_id", "station_no",
                       "station_name", "stationparameter_name",
                       "ts_unitsymbol", "parametertype_name", "ts_id")

together with the defaults station_latitude and station_longitude.

The decision to keep these has been a (deliberated) decision during the implementation based on the available information.

Still, more flexibility for the user (while keeping proper defaults) could improve the workflow. As an example, the API supports the md_returnfields: station_cartnorthing and station_carteasting as input as well, returning the coordinates in Lambert 72 instead of the default WGS84. Altough conversion can be done in R as well (see for example this function), the argument usage would simplify the workflow.

However, I do not have information or access to the possible arguments (either custom_attributes or md_returnfields) and the waterinfo documentation does not provide an overview as well. @WillemMaetens, is there such an overview available somewhere?

@stijnvanhoey stijnvanhoey changed the title Improved handling return_fields (i.e. columns in data.frame) Improved handling return_fields Nov 29, 2017
@WillemMaetens
Copy link
Collaborator

Info on the various call parameters and options can be found here: http://download.waterinfo.be/tsmdownload/KiWIS/KiWIS?datasource=0&service=kisters&type=queryServices&request=getrequestinfo

@stijnvanhoey
Copy link
Collaborator Author

stijnvanhoey commented Jan 28, 2019

@WillemMaetens is there an update of your previous link, as the link seems to be broken for the moment?

@stijnvanhoey
Copy link
Collaborator Author

stijnvanhoey commented Jun 24, 2019

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

No branches or pull requests

2 participants