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

feature: allow passing an explicit input resolution to read_csv #157

Merged

Conversation

Flix6x
Copy link
Collaborator

@Flix6x Flix6x commented Dec 6, 2023

This adds support for resampling input data with missing values - for which the original resolution cannot be inferred - with the original resolution explicitly set by the user (such as "PT5M").

@Flix6x Flix6x added the Resampling Dealing with downsampling and/or upsampling label Dec 6, 2023
@Flix6x Flix6x added this to the 1.26.0 milestone Dec 6, 2023
@Flix6x Flix6x self-assigned this Dec 6, 2023
@Flix6x Flix6x marked this pull request as ready for review December 6, 2023 00:33
@Flix6x Flix6x merged commit 4afa0de into fix/read_csv-with-upsampling Dec 11, 2023
5 checks passed
@Flix6x Flix6x deleted the feature/explicit-input-resolution branch December 11, 2023 16:33
Flix6x added a commit that referenced this pull request Jan 9, 2024
* fix: upsample by forward filling, but then also properly upsampling the last event

Signed-off-by: F.N. Claessen <[email protected]>

* style: black

Signed-off-by: F.N. Claessen <[email protected]>

* fix: function signature

Signed-off-by: F.N. Claessen <[email protected]>

* feature: allow passing an explicit input resolution to read_csv (#157)

Signed-off-by: F.N. Claessen <[email protected]>

* fix: let util function handle empty frame (methods using the util function already handled it)

Signed-off-by: F.N. Claessen <[email protected]>

* fix: raise if trying to upsample to instantaneous values

Signed-off-by: F.N. Claessen <[email protected]>

* fix: support for (multi-index) BeliefsDataFrames

Signed-off-by: F.N. Claessen <[email protected]>

* fix: raise if trying to upsample from instantaneous values

Signed-off-by: F.N. Claessen <[email protected]>

* deprecate: replace pad with ffill

Signed-off-by: F.N. Claessen <[email protected]>

* docs: explain use of L1 norm

Signed-off-by: F.N. Claessen <[email protected]>

---------

Signed-off-by: F.N. Claessen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resampling Dealing with downsampling and/or upsampling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants