Feedback on FluidPivotsPlugin

-- MariaDelCarmenHuertas - 07 Apr 2008

Thank you Maria and FluidSignal for contributing this plugin!

Some small feedback:

• It would be helpful to show one or two simple examples how this plugin is used. (in conjunction with the ChartPlugin if so)
• Usability: Avoid "this page" or "click here" links, see AvoidClickHereLinks
• Add the SHORTDESCRIPTION setting, the reports in the Plugins web depend on it. (I added it to the plugin topic)
• Add "Test if the installation was successful" explanation
• Use your name as the plugin author, e.g. TWiki:Main.MariaDelCarmenHuertas instead of TWiki:Main.FluidSignal; the Main.FluidSignal topic does not (and should not) exist. (I added it to the plugin topic)
• On copyright, use a direct link to the FluidSignal website. (I added it to the plugin topic)
• How about measuring and documenting the PluginBenchmarks numbers?
• Do not add the version umber to the package name, name it simply FluidPivotsPlugin.zip. (TWiki takes care of versioning; The reports in the Plugins web depend on this naming convention; I renamed the attachment)
• The plugin ships with lib/TWiki/Plugins/FluidPivotsPlugin/Table.pm, but the code refers to TWiki::Plugins::ChartPlugin::Table. Same for Parameters.pm. That is, those modules are never referenced; there is a dependency on ChartPlugin. Either remove those modules and document the dependency, or change the package names and references to TWiki::Plugins::FluidPivotsPlugin::Table
• Help on tag votes on the plugin topic so that people can discover it. (I added some tags)

-- PeterThoeny - 09 Apr 2008

This is good work - took some time for me to understand exactly what problem this Plugin solves. It would have been great help if the problem is stated clearly.

Agree with PeterThoeny - it always helps if a few sample/simple examples are explained/demonstrated with the Plugin.

I was able to install it successfully on my test instance, a Perl module CPAN:Math::Round is missing in the dependencies, corrected that in Plugin Topic.

Thanks for contributing the good work to the community-appreciated.

-- SopanShewale - 09 Apr 2008

Hm, can somebody explain briefly what it is this plugin does that ChartPlugin can't? I'm just curious why it was necessary to introduce a new plugin instead enhancing an existing one.

-- FranzJosefGigler - 09 Apr 2008