Comments for "SvgGrapher"

» Submit Your Comment
Please log in to enter your comment. If you are not a registered PEAR developer, you can comment by sending an email to pear-dev@lists.php.net.
» Comments
  • Guillaume Luchet  [2007-10-23 11:00 UTC]

    You can see the source code in the CVS repository at http://svggrapher.cvs.sourceforge.net/svggrapher/
  • Adam Harvey  [2007-10-23 11:35 UTC]

    I apologise if this seems snippy, as I mean it to be a genuine question: how is this package superior to the already extant Image_Graph package combined with the SVG output functionality in Image_Canvas?
  • Guillaume Luchet  [2007-10-23 14:28 UTC]

    Sorry, I didn't know about this package, SvgGrapher is just a "lighter" alternative promised to a great future in graph generation very hard world ;)

    Seriously, it provide a complete svg api to use all the svg capabilities like scripting, animation, filter effects, links...
  • Adam Harvey  [2007-10-23 16:27 UTC]

    Fair response. :)

    To be honest, I couldn't see myself voting for this package as things stand. Setting aside coding standard issues related to file and class naming and the like, neither of which I'm overly concerned about right now, I think there are two main issues that would be showstoppers for me:

    * The inclusion of what amounts to a complete XML generation library in XMLDocument.php and XMLElement.php; this seems to be unnecessary duplication of PHP5's DOM extension.

    * The duplication of effort between this and Image_Graph. I realise that there are features in this package that go above and beyond Image_Graph's capabilities -- the script.layer and script.tooltip plugins are things that could never be replicated in Image_Graph because of its use of Image_Canvas and underlying assumption that it's always generating a static image -- but I don't think they're compelling enough to justify having two separate graphing packages within PEAR (along with the confusion that sort of thing causes -- just look at the DB/MDB/MDB2 situation), particularly when one will have a narrow SVG-based focus.

    The SVG handling code is interesting, but perhaps not all that useful in isolation without validating inputs and the like, and that's a different package to this one anyway. :)

    Of course, that's all just my opinion. Other PEAR developers can (and probably will) disagree with me.

    One final comment which is completely unrelated to whether this is accepted into PEAR or not: there's a licensing issue in the package in that the Microsoft Web fonts in the tarball can't be legally redistributed as standalone .ttf files; they can only be redistributed in their original .exe or .sit.hqx forms, per http://corefonts.sourceforge.net/faq8.htm (which is a mirror of the original and now defunct Microsoft Web page dealing with font licensing). That is, of course, incredibly inconvenient in the finest Microsoft tradition. You may be able to look at using the Bitstream Vera or DejaVu fontset as a replacement, although you then run into possible issues with the client not necessarily having the fonts installed. Sorry, I know that sucks.
  • Tom Klingenberg  [2007-12-06 02:49 UTC]

    good work so far.
  • Tom Klingenberg  [2007-12-06 02:50 UTC]

    good work so far.
  • Tom Klingenberg  [2007-12-06 02:58 UTC]

    I feel sorry for comitting 2times. That was not intended even I think two times are not a problem at all. No problems about the m$ licensing issues, i think they should be solved faster than anyone can imagine, and if not i'am personally a supporter of fsfe who is willing to pay for any actions of the author that are necessary to publish this class under gpl v2 or it's succidors. it is fully dependend of the wishes of the author.
  • Tom Klingenberg  [2007-12-06 03:07 UTC]

    I feel sorry for comitting 2times. That was not intended even I think two times are not a problem at all. No problems about the m$ licensing issues, i think they should be solved faster than anyone can imagine, and if not i'am personally a supporter of fsfe who is willing to pay for any actions of the author that are necessary to publish this class under gpl v2 or it's succidors. it is fully dependend of the wishes of the author.
  • Tom Klingenberg  [2007-12-06 03:08 UTC]

    this submizzing akka rellllllloaddding fnction on pear.php.net suckz
  • Christian Weiske  [2009-06-25 08:04 UTC]

    Is this proposal dead, or are you going to finish it?