-
Notifications
You must be signed in to change notification settings - Fork 1
Avoid errors on array full nan values #4
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
Merged
Merged
Changes from all commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
a3dc413
avoid errors on DATA_DIR value
mano8 1f787c0
fix deprecated utcfromtimestamp
mano8 df24625
fix show graph
mano8 599f744
avoid errors on full nan values
mano8 3111533
Add noisy fina files to test
mano8 4e833f4
Add PyFina test notebook
mano8 f2a9944
add end_time value to getMeta function result
mano8 bd4ee7c
Fill ndarray with nan values to avoid arbitrary values
mano8 6d0ae2b
Avoid Reading file if time start read >= file end_time
mano8 d1c5794
End reading loop if pos out of bounds
mano8 c6c5c65
Apply changes
mano8 a078b0b
Correct end_time value
mano8 6fa2c2d
Apply codeRabit recomandations
mano8 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Large diffs are not rendered by default.
Oops, something went wrong.
Binary file not shown.
Binary file not shown.
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Avoid Exception
IndexError: index 0 is out of bounds for axis 0 with size 0,when
finiteness_obj.sum() = 0.from line 148
np.where(finiteness_obj)[0][0]Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mano8 : Just a wondering after integrating your datas in the tests : why do you record temperature (I guess it is temperature but maybe something else) each 1 seconds ? do you use a wired sensor ? I guess no otherwise you wouldn't have so many nan :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies for the file size. This file was generated from other files, with NaN values intentionally added for testing purposes. The source files contain solar battery voltage records sampled every second. They were created for testing in a different project. Feel free to exclude this file from the repository. :-)