Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix potential memory cache miss if scaling multiplier is slightly less than 1 due to rounding issue. #899

Merged
merged 2 commits into from
Oct 6, 2021

Conversation

colinrtwhite
Copy link
Member

Fixes: #817.

@gemiren
Copy link

gemiren commented Oct 4, 2021

This pull request seems to have this 1px issue fixed. Thanks.

@colinrtwhite colinrtwhite force-pushed the colin/fix_memory_cache_miss branch from 0c23f26 to cf3bfe3 Compare October 6, 2021 05:28
@colinrtwhite colinrtwhite merged commit 658514c into main Oct 6, 2021
@colinrtwhite colinrtwhite deleted the colin/fix_memory_cache_miss branch October 6, 2021 07:21
colinrtwhite added a commit that referenced this pull request Oct 5, 2022
…s than 1 due to rounding issue. (#899)

* Fix potential memory cache miss if scaling multiplier is slightly less than 1 due to rounding issue.

* Fix tests.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Memory cache is always missed
2 participants