-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Move MSYS2 to GitHub Actions #4642
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
Changes from 7 commits
0292910
479d583
a212225
d2f90d6
82204d5
c0f7a91
21325b8
d54fee9
adde693
0b6f909
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,6 +13,7 @@ | |
| assert_image_equal, | ||
| assert_image_similar, | ||
| assert_image_similar_tofile, | ||
| is_mingw, | ||
| is_pypy, | ||
| is_win32, | ||
| skip_unless_feature, | ||
|
|
@@ -660,6 +661,7 @@ def test_variation_get(self): | |
| {"name": b"Size", "minimum": 0, "maximum": 300, "default": 0} | ||
| ] | ||
|
|
||
| @pytest.mark.skipif(is_mingw(), reason="epsilon too high for meaningful test") | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure what is causing the difference, but I am getting epsilon around 33 for both variation tests. Checking manually, I see there is a significantly different kerning after the first letter, but the output looks correct to me. Note: MSYS2 is currently using FreeType version 2.10.2.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please could you attach images for each case?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sure. test_variation_set_by_name (epsilon 31.5340 on line 686 of PR): b.png (target - variation_adobe.png): test_variation_set_by_axes (epsilon 39.6856 on line 720 of PR): b.png (target - variation_adobe_axes.png): Basic and Raqm layout are pixel identical. In fact, it looks like there is a bug in the tests where the used |
||
| def test_variation_set_by_name(self): | ||
| font = self.get_font() | ||
|
|
||
|
|
@@ -692,6 +694,7 @@ def _check_text(font, path, epsilon): | |
| font.set_variation_by_name(name) | ||
| _check_text(font, "Tests/images/variation_tiny_name.png", 40) | ||
|
|
||
| @pytest.mark.skipif(is_mingw(), reason="epsilon too high for meaningful test") | ||
| def test_variation_set_by_axes(self): | ||
| font = self.get_font() | ||
|
|
||
|
|
||




Uh oh!
There was an error while loading. Please reload this page.