Skip to content

[graf2d] Test capability of creating png, jpeg and gif images#21560

Closed
dpiparo wants to merge 1 commit intoroot-project:masterfrom
dpiparo:testGif
Closed

[graf2d] Test capability of creating png, jpeg and gif images#21560
dpiparo wants to merge 1 commit intoroot-project:masterfrom
dpiparo:testGif

Conversation

@dpiparo
Copy link
Member

@dpiparo dpiparo commented Mar 10, 2026

A test, to see if gifs can be created on all platforms.
This PR was submitted after error in the creation of gif images occurred in this PR #21556
While searching the web, a user reported a very similar same issue https://root-forum.cern.ch/t/asimage2gif/60443, and the matter was never really followed up (@couet)

TH1D h("myHisto", "The Title;the X;the Y", 64, -4, 4);
h.FillRandom("gaus");
h.Draw();
std::vector<std::string> fileNames{"f.jpeg", "f.png", "f.gif"};
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
std::vector<std::string> fileNames{"f.jpeg", "f.png", "f.gif"};
std::vector<std::string> fileNames{"f.jpeg", "f.png", "f.gif", "f.bmp"};

@@ -0,0 +1,4 @@
Info in <TCanvas::Print>: jpg file f.jpeg has been created
Info in <TCanvas::Print>: png file f.png has been created
Info in <TCanvas::Print>: gif file f.gif has been created
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Info in <TCanvas::Print>: gif file f.gif has been created
Info in <TCanvas::Print>: gif file f.gif has been created
Info in <TCanvas::Print>: gif file f.bmp has been created

@github-actions
Copy link

Test Results

    23 files      23 suites   3d 7h 35m 9s ⏱️
 3 820 tests  3 818 ✅ 1 💤  1 ❌
79 344 runs  79 324 ✅ 9 💤 11 ❌

For more details on these failures, see this check.

Results for commit bc1a1b0.

@dpiparo dpiparo closed this Mar 11, 2026
@dpiparo dpiparo deleted the testGif branch March 11, 2026 19:43
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.

2 participants