Phoronix Forums  

Go Back   Phoronix Forums > Phoronix > Phoronix Test Suite

Phoronix Test Suite Discussion & collaboration on the Phoronix Test Suite software and specification.

Reply
 
Thread Tools Display Modes
  #1  
Old 05-08-2008, 03:49 PM
uncle_fungus uncle_fungus is offline
Phoronix Test Suite
 
Join Date: Apr 2008
Location: York, UK
Posts: 134
Default Question about results graphs

I've noticed that the png files that are produced by pts-grapher are rather large (170Kb per file) and was wondering why you aren't applying any of png's compression to the images.

It only needs a 1 character change in pts_Graph.php to reduce size of the generated images from 170Kb to 6Kb each. I suspect this would be useful to have on PTS Global to as it would significantly reduce the bandwidth of viewing test results.

Code:
diff --git a/pts-core/objects/pts_Graph.php b/pts-core/objects/pts_Graph.php
index 0c55ff2..f58b189 100644
--- a/pts-core/objects/pts_Graph.php
+++ b/pts-core/objects/pts_Graph.php
@@ -438,7 +438,7 @@ class pts_Graph
        }
        protected function return_graph_image()
        {
-               Imagepng($this->graph_image, $this->graph_output, 0);
+               Imagepng($this->graph_image, $this->graph_output, 9);
                ImageDestroy($this->graph_image);
        }
        public function save_graph($file)
Reply With Quote
  #2  
Old 05-08-2008, 03:55 PM
Michael Michael is offline
Phoronix
 
Join Date: Jun 2006
Location: United States
Posts: 4,559
Default

Yeah, I really don't know why I hadn't enabled PNG compression earlier... Anyways, it's compressed now in git.
Reply With Quote
Reply

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT -5. The time now is 05:27 AM.


Powered by vBulletin® Version 3.8.4
Copyright ©2000 - 2009, Jelsoft Enterprises Ltd.
Copyright ©2004 - 2009 by Phoronix Media.