-
Notifications
You must be signed in to change notification settings - Fork 301
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
fix: Handle 'inf' in _convert_numeric #1449
fix: Handle 'inf' in _convert_numeric #1449
Conversation
Reading data from the sqlite database goes through a converter (_convert_numeric) in qcodes/dataset/sqlite_base.py that now can handle 'inf' as the numeric value properly. Previously, I threw an error because it incorrectly assumed that 'inf' would be an integer.
Codecov Report
@@ Coverage Diff @@
## master #1449 +/- ##
=========================================
+ Coverage 73.8% 73.8% +<.01%
=========================================
Files 92 92
Lines 10444 10446 +2
=========================================
+ Hits 7708 7710 +2
Misses 2736 2736 |
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.
This looks great!
Could you add a test for it (smth similar to this https://github.com/QCoDeS/Qcodes/blob/a3113ee98f8522490f90578236b0b92eb7763999/qcodes/tests/dataset/test_dataset_basic.py#L580-L587) ?
…m/simonzihlmann/Qcodes into fix/handle_inf_in_convert_numeric Update local version of branch
Test that we can insert and retrieve numpy inf in the data set
Reading data from the sqlite database goes through a converter (_convert_numeric) in qcodes/dataset/sqlite_base.py that now can handle 'inf' as the numeric value properly. Previously, it threw an error because it incorrectly assumed that 'inf' would be an integer.
@astafan8