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

Helpers: precision improvements #81

Open
wants to merge 2 commits into
base: master
from

Conversation

@sohkai
Copy link
Member

commented Oct 9, 2019

cc @schmidsi.

Improves the precision changes, such that any truncation to the fraction results in a ~.

Also makes the same fraction check more robust than the previous one.

sohkai added 2 commits Oct 9, 2019
@sohkai sohkai requested a review from izqui Oct 9, 2019
@coveralls

This comment has been minimized.

Copy link

commented Oct 9, 2019

Coverage Status

Coverage increased (+0.009%) to 91.228% when pulling f0c32b9 on precision-improvements into 35a446d on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.