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.

Sign up for GitHub

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

Coercing timedelta to float #89

Merged

alxmrs merged 2 commits into main from mv-float-json

Jan 18, 2022

Collaborator

alxmrs commented Jan 18, 2022 •

edited

Loading

Fixes #81.

Added tests to be sure that no values are np.float64s
Found and fixed the JSON serialization error.

alxmrs added 2 commits

January 17, 2022 10:41


Added check to ensure output is not a numpy type.

8f8cad4


JSON Error Fix: Converting timedelta to float.

95fd2f7

alxmrs requested review from pramodg, mt467 and uhager

January 18, 2022 00:48

pramodg approved these changes

View reviewed changes

Collaborator Author

alxmrs commented Jan 18, 2022

Thanks for the holiday review, @pramodg! 😃

alxmrs merged commit 8526867 into main

alxmrs deleted the mv-float-json branch

January 18, 2022 01:17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment