Skip to content

[graphics] fix different SavePrimitve problems#20295

Merged
linev merged 10 commits intoroot-project:masterfrom
linev:fix_save_primitive
Nov 6, 2025
Merged

[graphics] fix different SavePrimitve problems#20295
linev merged 10 commits intoroot-project:masterfrom
linev:fix_save_primitive

Conversation

@linev
Copy link
Member

@linev linev commented Nov 5, 2025

  1. Default values for fill attributes were not correct in many places. So one has to specify -1, -1 in SaveFillAttributes to correctly store them
  2. For line attributes check if gStlye modified and force saving (1,1,1) attributes if this is a case
  3. Correctly save frame border mode in canvas
  4. For TMultiGraph store histogram like for TGraph - to avoid complex situation when gPad->Update() called many time during execution of saved macro
  5. Update JSROOT where axis ticks error is fixed now.
  6. Update stressGraphics_web.ref file after JSROOT fixes
  7. Adjust hist tests where SavePrimitive is testes - fill attributes now forced.

@github-actions
Copy link

github-actions bot commented Nov 5, 2025

Test Results

    19 files      19 suites   3d 3h 31m 22s ⏱️
 3 707 tests  3 707 ✅ 0 💤 0 ❌
68 966 runs  68 966 ✅ 0 💤 0 ❌

Results for commit 25d764e.

♻️ This comment has been updated with latest results.

linev added 9 commits November 5, 2025 16:16
If no exact init values defined for TAttFill, gStyle configuration is used.
So one always have to save fill attributes for such classes
If no exact init values defined for TAttFill, gStyle configuration is used.
So one always have to save fill attributes for such classes
If no exact init values defined for TAttFill, gStyle configuration is used.
So one always have to save fill attributes for such classes
Allows to configure histogram used for axes drawing
Like in TGraph, store histogram which used for axis painting.
Resolves problem that GetHistogram or GetAxis methods force recreation
of histogram object and updating the pad.
1. Implement for `TPie` 3d, text, title drawing including interactivity
2. Fix - proper paint axis labels on both sides when pad.fTickx/y = 2
3. Fix - paint frame border mode/size from TCanvas
After fixes in JSROOT several tests works better now
@linev linev force-pushed the fix_save_primitive branch from ef34c98 to b21b198 Compare November 5, 2025 15:23
@linev linev merged commit fa74d4f into root-project:master Nov 6, 2025
23 of 27 checks passed
@linev linev deleted the fix_save_primitive branch November 6, 2025 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants