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

Coerce coops results to numeric, account for data results not named "data", add examples #129

Merged
merged 4 commits into from
Jan 2, 2016

Conversation

jsta
Copy link
Contributor

@jsta jsta commented Dec 29, 2015

#126 Still unsure if I have coerced numeric results in the best way. The only consistent numeric column across all products is the second column. Here is a breakdown of the various column types:

18 col

all numeric

monthly_mean

5 col

posix-date, numeric, numeric, character, character

water_level

3 col

posix-date, numeric, character; #only for great lakes stations

daily_mean

posix-date, numeric, character

air_temperature

posix-date, numeric, character

water_temperature

posix-date, numeric, character

air_pressure

posix-date, numeric, character

conductivity

posix-date, numeric, numeric

salinity

posix-date, numeric, character

humidity

6 col

posix-date, numeric, numeric, character, numeric, character

wind

4 col

posix-date, numeric, numeric, character

hourly_height

posix-date, numeric, character, character

high_low

posix-date and numerics

currents

2 col

posix-date and numeric

one_minute_water_level

character and numeric

datums

posix-date & numeric

predictions

#no data at any station?
air_gap

visibility

#' # Get air temperature at Vaca Key (2723970)
#' # Get daily mean water level data at Fairport, OH (9063053)
#' coops_search(station_name = 9063053, begin_date = 20150927, end_date = 20150928,
#' product = "daily_mean", datum = "stnd"), time_zone = "lst")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there's an extra parentheses here datum = "stnd"), time_zone = "lst")

@sckott
Copy link
Contributor

sckott commented Dec 29, 2015

can you fix that one thing above, otherwise, looks good

sckott added a commit that referenced this pull request Jan 2, 2016
Coerce coops results to numeric, account for data results not named "data", add examples
@sckott sckott merged commit c47a75d into ropensci:master Jan 2, 2016
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

Successfully merging this pull request may close these issues.

2 participants