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

TRatioplot gives "different" results with Web Graphics #14842

Closed
1 task done
couet opened this issue Feb 27, 2024 · 4 comments · Fixed by #14850 or #14905
Closed
1 task done

TRatioplot gives "different" results with Web Graphics #14842

couet opened this issue Feb 27, 2024 · 4 comments · Fixed by #14850 or #14905
Assignees
Labels

Comments

@couet
Copy link
Member

couet commented Feb 27, 2024

Check duplicate issues.

  • Checked for duplicates

Description

Run any TRatioPlot examples and compare the result of web graphics and standard graphics. The result is different.

Reproducer

Any TRatioplot example in tutorials.

ROOT version

Any

Installation method

any

Operating system

all

Additional context

No response

@couet couet added the bug label Feb 27, 2024
@couet couet changed the title TRatioplot gives a different results with Web Graphics TRatioplot gives "different" results with Web Graphics Feb 27, 2024
@linev
Copy link
Member

linev commented Feb 28, 2024

When I run standard demos, I also see difference.
But if I call gRandom->SetSeed(1) before filling histograms, I get similar results.

ratioplot1.C with normal graphics:

C

ratioplot1.C with web graphics:

Canvas

There is difference in axes labels size, but this I want to investigate.

@linev
Copy link
Member

linev commented Feb 28, 2024

Also labels sizes are clarified - now I understand how TRatioPlot axis drawing implemented in ROOT.
I will submit PR which fixes several small issues around TRatioPlot

@linev
Copy link
Member

linev commented Feb 29, 2024

There are still some differences, need to improve

Copy link

github-actions bot commented Mar 8, 2024

Hi @linev,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

@linev linev added this to Issues in Fixed in 6.32.00 via automation Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
2 participants