Opened 13 years ago

Closed 10 years ago

#97 closed task (fixed)

Gnuplot is very slow, replace it with plplot

Reported by: Jari Häkkinen Owned by: Jari Häkkinen
Priority: blocker Milestone: svndigest 0.8
Component: core Version: trunk
Keywords: Cc:

Description (last modified by Jari Häkkinen)

Needed for #75 and #279

Depends on plplot bug http://sourceforge.net/tracker/?func=detail&aid=2901388&group_id=2915&atid=102915 Until the plplot issue is resolved you must download plplot 5.9.5 and patch the plstream destructor manually before compiling and installing plplot.

Gnuplot replaced with plplot, the rest of the tickets are split into #402, #403, #404, and #405

Attachments (2)

plstream.cc (59.5 KB) - added by Jari Häkkinen 10 years ago.
Taken from plplot subversion trunk revision 10626
plstream.h (36.9 KB) - added by Jari Häkkinen 10 years ago.
Taken from plplot subversion trunk revision 10626

Download all attachments as: .zip

Change History (32)

comment:1 Changed 13 years ago by Jari Häkkinen

Milestone: 0.7

comment:2 Changed 12 years ago by Jari Häkkinen

Priority: majorblocker

comment:3 Changed 12 years ago by Jari Häkkinen

There is no API for the current gnuplot and no plans to create one exist

comment:4 Changed 12 years ago by Peter Johansson

What about GNU libplot?

http://www.gnu.org/software/plotutils/

There is clearly an API for C/C++

comment:5 Changed 12 years ago by Peter Johansson

Description: modified (diff)

added relataion with #75

comment:6 Changed 11 years ago by Jari Häkkinen

Status: newassigned

comment:7 Changed 11 years ago by Jari Häkkinen

This change will be implemented in branch source:branches/replacing_gnuplot

comment:8 in reply to:  4 ; Changed 10 years ago by Jari Häkkinen

Replying to peter:

What about GNU libplot?

http://www.gnu.org/software/plotutils/

There is clearly an API for C/C++

I looking at http://plplot.sourceforge.net/

comment:9 in reply to:  8 Changed 10 years ago by Jari Häkkinen

Replying to jari:

Replying to peter:

What about GNU libplot?

http://www.gnu.org/software/plotutils/

There is clearly an API for C/C++

I looking at http://plplot.sourceforge.net/

Look at the example page, http://plplot.sourceforge.net/examples.php, click on one of the sample plots and select your favourite language in the tab, copy&paste! That is it, I am going for plplot and I am starting tomorrow so there is some time to try to convince me for something else.

comment:10 Changed 10 years ago by Peter Johansson

Found that plplot is included in Fedora 9, the examples look good, and it is licensed under LGPL, so go ahead.

comment:11 Changed 10 years ago by Jari Häkkinen

(In [786]) Addresses #97. Removed gnuplot checks in configure.ac and added checks for PLplot. Updated README.

comment:12 Changed 10 years ago by Jari Häkkinen

(In [787]) Addresses #97. Removed gnuplot tests.

comment:13 Changed 10 years ago by Jari Häkkinen

(In [788]) Addresses #97. Removed gnuplot files, added Graph class.

comment:14 Changed 10 years ago by Jari Häkkinen

(In [789]) Addresses #97. Removed GnuplotFE dependency in File.cc

comment:15 Changed 10 years ago by Peter Johansson

I think it should be possible to build and run svndigest without the plot machinery (plplot). 1) svndigest --copyright is still useful 2) Possibly we should even allow reports without images (just a [plot] instead).

comment:16 Changed 10 years ago by Peter Johansson

Milestone: svndigest 0.7svndigest 0.8

comment:17 Changed 10 years ago by Jari Häkkinen

(In [855]) Addresses #97. One summary plot is generated with plplot ... much more to do.

comment:18 Changed 10 years ago by Jari Häkkinen

(In [858]) Addresses #97. Merging non-gnuplot plot generation code. Gnuplot traces left and lot of work before publication quality plots are generated.

comment:19 Changed 10 years ago by Jari Häkkinen

Description: modified (diff)
Summary: Gnuplot is very slow, either compile gnuplot into binary or find something newGnuplot is very slow, replace it with plplot

comment:20 Changed 10 years ago by Jari Häkkinen

(In [859]) Addresses #97. Improvements to summary plot.

comment:21 Changed 10 years ago by Jari Häkkinen

(In [863]) Addresses #97. Fixed background, added doxygen and other comments.

comment:22 Changed 10 years ago by Jari Häkkinen

(In [864]) Addresses #97. All plots generated. Now its time to go for publication quality plots.

comment:23 Changed 10 years ago by Jari Häkkinen

Description: modified (diff)

comment:24 Changed 10 years ago by Jari Häkkinen

Description: modified (diff)

comment:26 Changed 10 years ago by Jari Häkkinen

I'll look into that. My suggested patch works for us since we never have more than one stream open at any time. However, we should of course use the official fix. I'll create a patch for released plplot 5.9.5 and post it somewhere. I prefer not to live on plplot trunk.

Changed 10 years ago by Jari Häkkinen

Attachment: plstream.cc added

Taken from plplot subversion trunk revision 10626

Changed 10 years ago by Jari Häkkinen

Attachment: plstream.h added

Taken from plplot subversion trunk revision 10626

comment:27 Changed 10 years ago by Jari Häkkinen

I attached plstream.cc and plstream.h to this ticket. These files are copied from the plplot subversion trunk revision 10626. It was too hard to create a patch due to many avoidable changes in the files (space, change of comment style, moves of '{' from one line to another) so I attached the complete files. I tested them with svndigest and make check passes and plots are generated.

comment:28 Changed 10 years ago by Jari Häkkinen

(In [917]) Addresses #97. Background white and transparent.

comment:29 Changed 10 years ago by Jari Häkkinen

(In [945]) Addresses #97. This branch was merged into the trunk and is not needed anymore.

comment:30 Changed 10 years ago by Jari Häkkinen

Resolution: fixed
Status: assignedclosed

(In [946]) Fixes #97. Well, gnuplot is now removed and sub task are defined in other tickets.

Note: See TracTickets for help on using tickets.