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

tests-d2d fonts #16

Open
JohnDog3112 opened this issue Aug 20, 2024 · 1 comment · Fixed by #18
Open

tests-d2d fonts #16

JohnDog3112 opened this issue Aug 20, 2024 · 1 comment · Fixed by #18
Labels
bug Something isn't working

Comments

@JohnDog3112
Copy link
Collaborator

JohnDog3112 commented Aug 20, 2024

Tests relating to text seem to output different hashes for Windows and Linux during testing.

Possible problems

  • Different Default Fonts
    • Windows and Linux has a different set of fonts
    • However, trying to load the same fonts on Linux, the fonts still differ, so either this isn't the only issue, or the fonts aren't exactly the same.
  • GPU/GPU Driver differences
    • This seems to be most likely. I just tested it on my laptop (also running Linux) and the test hashes were different from what I got when running it on my desktop.
      • Laptop: Optimus drivers for NVIDIA GTX 1050ti GPU and Intel i7 8th gen CPU with integrated graphics
      • Desktop: NVIDIA Drivers for NVIDIA RTX 3050

Since the issue seems like it might be due to the GPU/GPU Drivers, I'm not quite sure how to fix this test case other than only running it in a VM with standard drivers.

@JohnDog3112 JohnDog3112 added the bug Something isn't working label Aug 20, 2024
@twiddlingbits
Copy link
Owner

twiddlingbits commented Aug 21, 2024

Leave the issue open. Maybe updated the test output so that the "known won't work on other platforms" tests are grouped together with a heading like "These may fail on platforms others than those the test result CRC was generated for". Then, you can make sure they keep working for you. We could also add a second set of CRCs, mine, so at least you and me can run the tests. I think in a professional environment, they would probably set up different dedicated test machines. We are not going to do that. I can't think of a generic solution.

But other than the above, lets punt, and leave the issue open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants