Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Serialising Inf float produces invalid JSON #1423

Closed
SimonSchneider opened this issue Feb 28, 2023 · 0 comments · Fixed by #1427
Closed

Serialising Inf float produces invalid JSON #1423

SimonSchneider opened this issue Feb 28, 2023 · 0 comments · Fixed by #1427
Labels
no-changelog Issues whose changes are covered by a PR and thus should not be shown in the changelog

Comments

@SimonSchneider
Copy link
Contributor

Similarly to: #988 the json produced when serialising a float with inf produces invalid JSON with { "xyz": inf } which is not in the json standard.

I'd be happy to fix this similarly to how it was fixed for NaNs in #990 . If that is the route to go.
@SimonSchneider SimonSchneider mentioned this issue Mar 7, 2023
Merged
@jorgecarleitao jorgecarleitao added bug Something isn't working no-changelog Issues whose changes are covered by a PR and thus should not be shown in the changelog and removed bug Something isn't working labels Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
no-changelog Issues whose changes are covered by a PR and thus should not be shown in the changelog
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Changed encoded float::Inf as null in json SimonSchneider/arrow2
2 participants
@jorgecarleitao @SimonSchneider