Skip to content
This repository was archived by the owner on Mar 25, 2024. It is now read-only.

Serialize f32 #214

Merged
merged 2 commits into from
Sep 10, 2021
Merged

Serialize f32 #214

merged 2 commits into from
Sep 10, 2021

Conversation

cheradenine
Copy link

This change adds explicit formatting for f32 to avoid artifacts when promoting to f64.

Fixes #213

john.burkhardt added 2 commits September 10, 2021 11:59

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Copy link
Owner

@dtolnay dtolnay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@dtolnay dtolnay merged commit 045dfed into dtolnay:master Sep 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Serialization of f32 can produce rounding that is hard for humans to read
2 participants