Updated logging | Updated timeseries flags | Excel-style dates | Removal of records with invalid location data | Updated README#7
Conversation
This will be reported in the logs and warnings so that the user can check the specific record at the source and the developer can account for that situation if desired.
Much of the dates for the OSE Roswell data are in Excel date number format. This commit adds a check for this format and converts it to a standard datetime format.
--unified_timeseries for a single file of timeseries data --separated_timeseries for when each locations has a timeseries file
All logs and warnings are now handled by the logger
This is an updated README.md that pertains to the current state of the DIE. I plan to make some changes and when that is done I will update the README furtehr
Still in version 0, but bumping the minor version because of how the different timeseries are called for
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated Logging
Updated timeseries flags
--separated_timeseriesand--unified_timeseries. The former creates individual files for every site, whereas the latter creates one unified file for all sitesExcel-style dates
Removal of records with invalid location data
Updated README
--bboxneeded to be input as--bbox 'x1 y1, x2 y2--start_dateandend_dateshould be input asYYYY-MM-DD