-
Notifications
You must be signed in to change notification settings - Fork 31
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
Generalize read_field_obs, resample_to_field, and plot_var_compare #191
Conversation
x = as.numeric(as.POSIXct(data$DateTime)) | ||
y = data$Depth | ||
z = data$Observed_wTemp | ||
z = data[,paste0('Observed_', var_name)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
didn't know that data[, paste0('Observed','_temp')]
is the same as data[[paste0('Observed_', var_name)]]
.
Looks good to me. It would be nice to add a test to verify this working across variables though. |
Those files have all lines changed because they were in the old weirdo mac format with only \CR as a newline character (which is why the old files show up in github as being all one line). I've switched them all to *nix format. |
Failures are github 403's. Tests added. Merging. |
Generalize read_field_obs, resample_to_field, and plot_var_compare
Goal is to use this comparison functionality on any parameter. These changes currently break backward compatibility with field data using "wtemp" header. This also changes output of
get_var
to no longer bewtr_
andelev_
, now istemp_
for temp (generalized tovarname_
based on parameter passed) andvarname.elev_
for any bottom referenced data.We may want to maintain more backwards compatibility, but that does make the other functionality harder to implement as it is easy to assume the header is
varname
andvarname_depth
as opposed to sometimes temp being namedwtr
orwtemp
.