Skip to content
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

Add hex representation of floats in DESYNC output #52

Merged
merged 2 commits into from
Mar 24, 2024

Conversation

malleoz
Copy link
Contributor

@malleoz malleoz commented Feb 22, 2024

Shows the hex representation of floats in the DESYNC output. Also avoid imprecise printing of the numeric representation by using %g instead of %f.

I created a helper function since we will likely have to perform this operation very often for debugging.

@vabold
Copy link
Owner

vabold commented Mar 23, 2024

To my understanding, %g has the same level of precision as %f, except %e can be introduced if the representation is shorter.

@malleoz
Copy link
Contributor Author

malleoz commented Mar 23, 2024

Hmm yeah not sure why I wrote that. %g guarantees that the string representation is the minimum string required to precisely represent the number

@malleoz malleoz force-pushed the hex_logging branch 2 times, most recently from 7388ad6 to 9c1d66c Compare March 24, 2024 04:12
@vabold
Copy link
Owner

vabold commented Mar 24, 2024

Failing build is due to the updated GitHub action not being run until after merge.

@vabold vabold merged commit 431cf39 into vabold:main Mar 24, 2024
2 of 3 checks passed
@malleoz malleoz deleted the hex_logging branch March 31, 2024 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants