Skip to content

Adjust test/stressGraphics.ref #17398

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

Merged
merged 1 commit into from
Apr 12, 2025

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

github-actions bot commented Jan 10, 2025

Test Results

    17 files      17 suites   4d 1h 36m 20s ⏱️
 2 693 tests  2 683 ✅ 0 💤 10 ❌
44 127 runs  44 116 ✅ 0 💤 11 ❌

For more details on these failures, see this check.

Results for commit b0f07c6.

♻️ This comment has been updated with latest results.

@guitargeek guitargeek merged commit 3117fcb into root-project:master Apr 12, 2025
17 of 20 checks passed
@ellert ellert deleted the stressGraphics.ref branch April 19, 2025 03:37
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.

4 participants