Skip to content

Fix test failures after skia-pathops m113 to m143 update#331

Merged
anthrotype merged 1 commit intomainfrom
update-tests-for-skia-m143
Nov 11, 2025
Merged

Fix test failures after skia-pathops m113 to m143 update#331
anthrotype merged 1 commit intomainfrom
update-tests-for-skia-m143

Conversation

@anthrotype
Copy link
Member

After updating skia-pathops from m113 to m143 in fonttools/skia-pathops#81, 5 tests were failing on macOS due to floating-point precision differences in stroke expansion. These differences arise from cross-platform floating-point variations (macOS ARM vs Linux x86_64) and minor algorithmic improvements in skia-pathops m143. Rather than regenerating all expected files, we round both actual and expected output to the minimum precision needed for each test.

After updating skia-pathops from m113 to m143 in fonttools/skia-pathops#81, 5 tests were failing on macOS due to floating-point precision differences in stroke expansion.
These differences arise from cross-platform floating-point variations (macOS ARM vs Linux x86_64) and minor algorithmic improvements in skia-pathops m143.
Rather than regenerating all expected files, we round both actual and
expected output to the minimum precision needed for each test.
@anthrotype anthrotype merged commit 47e9dd0 into main Nov 11, 2025
9 checks passed
@anthrotype anthrotype deleted the update-tests-for-skia-m143 branch November 11, 2025 22:50
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.

1 participant