Skip to content

Adjust test/stressGraphics.ref #18989

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

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

Conversation

ellert
Copy link
Contributor

@ellert ellert commented Jun 7, 2025

Test 37: TScatter with TPolyMarker test............................ OK
         PDF output......................................... 37 FAILED
         Result    = 22236
         Reference = 22560
         Error     = 324 (was 300)
         JPG output................................................ OK
         PNG output................................................ OK
         C file result............................................. OK

Test 37: TScatter with TPolyMarker test............................ OK
         PDF output......................................... 37 FAILED
         Result    = 22236
         Reference = 22560
         Error     = 324 (was 300)
         JPG output................................................ OK
         PNG output................................................ OK
         C file result............................................. OK
@ellert ellert requested a review from dpiparo as a code owner June 7, 2025 15:10
@dpiparo
Copy link
Member

dpiparo commented Jun 7, 2025

Thanks for this changes. Could you give some details about the problem you are trying to address?

Copy link

github-actions bot commented Jun 7, 2025

Test Results

    19 files      19 suites   3d 8h 29m 31s ⏱️
 2 805 tests  2 805 ✅ 0 💤 0 ❌
51 795 runs  51 795 ✅ 0 💤 0 ❌

Results for commit 8128680.

@couet
Copy link
Member

couet commented Jun 10, 2025

On which machine to you see this issue ? i do not see it on my mac. I add @linev in the loop because he did the las chnages in stressGrpahics

@couet couet requested a review from linev June 10, 2025 06:41
@guitargeek
Copy link
Contributor

@ellert, ping on this. Before we merge, could you please tell us here on which platform and architecture you saw this failure? Just for the record.

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.

5 participants