rvasa / jseat

Automatically exported from code.google.com/p/jseat
0 stars 0 forks source link

Update the ColurMapDecorator so that it can be used as a truly generic visualization #13

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Background:
Visualizations (that make use of ReportDecorators) should generally be a 
generic approach. An example of one which isn't would be the 
ColourMapDecorator.

Currently it only works with the ModificationIntensityReportVisitor. 
Theoretically, it should be able to produce a colour map for any histogram 
dataset. When a report contains mixed column types, it should be able to 
specify which columns are included for colour mapping.

Because it currently only works with the ModifcationIntensityReportVisitor 
it assumes all values in the data set are between -1 and 1. These ranges 
should be able to be customised to suit other metric types. This is 
actually controlled by the PaintBasedColourGenerator utility.

The x/y axis is currently hardcoded as "RSN" and "RSN Born" respectively.

Issue:
Update the ColurMapDecorator so that it can be used as a truly generic 
visualization.

Add ability to specify which columns are included when the decorator 
creates its dataset (when createXYZDataset(MetricTable mt) is called).

Allow x/y axis labels to be customised.

Update the PaintBasedColourGenerator utility to allow customisation of the 
range values when generating a paint scale. It currently supports:

// Generates a LookupPaintScale for the specified IntensityStyle
// with the specified number of shades.
static LookupPaintScale generatePaintScale(IntensityStyle style, int max)

Could be changed to something like...

// minValue The minimum range
// maxValue The maximum range
// shades The number of shades to generate
// maxShadeValue The maximum value that shades should be generated within
static LookupPaintScale generatePaintScale(IntensityStyle style, int 
minValue, int maxValue, int shades, int maxShades)

Original issue reported on code.google.com by jtha...@gmail.com on 17 Jul 2007 at 6:31