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

Filling Missing dates functionality #16

Open
volloholic opened this issue May 20, 2015 · 3 comments
Open

Filling Missing dates functionality #16

volloholic opened this issue May 20, 2015 · 3 comments

Comments

@volloholic
Copy link
Member

missing_dates_check in the climate_data object is currently only functional for Daily data needs to be implemented for other time periods.

@rposn
Copy link
Contributor

rposn commented May 26, 2015

"date_col_check" method in "climate_data_refclass.R" is also only functional for Daily (see attached). It converts the date/time column into a date column, since uses the "as.Date" function. This function takes into account only the dates and discards the times. This should be modified for "subdaily" data...
date_col_check

@dannyparsons
Copy link
Contributor

The section below that (see attached) creates the date_time column for subdaily data. So subdaily data has a date column with "Date" class (no time) and a date_time column with "POSIXct" class. So only the date column discards the time.
I think David mentioned before the reasons we decided to have a separate date and date_time column because of rare issues with time zones.
For subdaily data, the date column may not be needed, since this information is already contained in the date_time column, but for consistency we decided that every climate data data frame must have a date column stored in a consistent format.
We should however, have an else if statement, below the if statement in your screen shot, that creates the date column if a date_time column is present and the date column does not already exist. This should be separate issue now I think, as this issue started by David is about missing dates.
clipboard01

@rposn
Copy link
Contributor

rposn commented May 26, 2015

Thanks for the reply. I will open then a new issue under the title "subdaily dataset"

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

3 participants