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

Better NaN/error handling #18

Open
tilsche opened this issue Aug 19, 2021 · 1 comment
Open

Better NaN/error handling #18

tilsche opened this issue Aug 19, 2021 · 1 comment
Labels
wontfix This will not be worked on

Comments

@tilsche
Copy link
Contributor

tilsche commented Aug 19, 2021

There might be a reasonable way to handle NaNs:

  • Record them as raw values
  • But don't include them in any of the aggregates
  • Reduce the active time for the interval before the NaN value
@bmario bmario added the wontfix This will not be worked on label Sep 30, 2021
@bmario
Copy link
Member

bmario commented Sep 30, 2021

After a short discussion, the problem here is that count needs to be dealt with as well, as it is used for indexing AND calculation of the mean value. A possible solution would be a nan_count, but that is a major breaking change. Especially, the file format would change.

@bmario bmario closed this as completed Sep 30, 2021
@bmario bmario reopened this Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants