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

Adjust test/stressGraphics.ref #17398

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ellert
Copy link
Contributor

@ellert ellert commented Jan 9, 2025

This Pull request:

Changes or fixes:

One of the tests fails with just a few bytes on latest Fedora releases. Failures can be seen for both test-stressgraphics and test-stressgraphics-interpreted. Failures are reported for sizes being 201, 202 or 203 bytes larger than the reference, with thw limit currently set to 200 bytes.

Test 19: TGaxis 3 (Time on axis)................................... OK
         PDF output......................................... 19 FAILED
         Result    = 42723
         Reference = 42926
         Error     = 203 (was 200)
         JPG output................................................ OK
         PNG output................................................ OK
         C file result............................................. OK

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

One of the tests fails with just a few bytes on latest Fedora
releases. Failures can be seen for both test-stressgraphics and
test-stressgraphics-interpreted. Failures are reported for sizes being
201, 202 or 203 bytes larger than the reference, with thw limit
currently set to 200 bytes.

Test 19: TGaxis 3 (Time on axis)................................... OK
         PDF output......................................... 19 FAILED
         Result    = 42723
         Reference = 42926
         Error     = 203 (was 200)
         JPG output................................................ OK
         PNG output................................................ OK
         C file result............................................. OK
@ellert ellert requested a review from dpiparo as a code owner January 9, 2025 21:27
@bellenot bellenot requested a review from couet January 10, 2025 07:27
@couet
Copy link
Member

couet commented Jan 10, 2025

The latest mods in that code were made by @linev . let me add him in the loop.

@couet couet requested review from linev and removed request for dpiparo January 10, 2025 07:32
Copy link

Test Results

    17 files      17 suites   4d 1h 43m 41s ⏱️
 2 693 tests  2 689 ✅ 0 💤 4 ❌
44 127 runs  44 123 ✅ 0 💤 4 ❌

For more details on these failures, see this check.

Results for commit b0f07c6.

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.

3 participants