Enhance read csv #54
Merged
Enhance read csv #54
Conversation
This file contains 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
nhoening
approved these changes
Apr 6, 2021
nhoening
reviewed
Apr 13, 2021
| and the second column has to contain the event values. | ||
| You also need to pass explicit values for the belief horizon/time and cumulative probability, | ||
| in addition to the sensor and source. | ||
| Useful ddditional kwargs passed to pandas are parse_dates=True and infer_datetime_format=True. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Add support to read in a simple time series (two columns) from CSV, provided the caller passes a source and belief time/horizon explicitly (besides the sensor). Not sure if using
len(df.columns) == 2is the best way to distinguish this (arguably common) data format. Should I rather add an explicit boolean option to the function?