[Dive4elements-commits] [PATCH] XYChartGenerator: Avoid "range merging" for area series, as these might extend

Wald Commits scm-commit at wald.intevation.org
Thu Dec 6 17:15:22 CET 2012


# HG changeset patch
# User Felix Wolfsteller <felix.wolfsteller at intevation.de>
# Date 1354795827 -3600
# Node ID 31327b3b2455b42690b59a9a3b32e47f53777933
# Parent  64664e8244ff2260b4c0d89e442e722e4e64e112
XYChartGenerator: Avoid "range merging" for area series, as these might extend
to fake "infinity" in case of showing an area "above axis". Explain this in code
and remove a TODO.

diff -r 64664e8244ff -r 31327b3b2455 flys-artifacts/src/main/java/de/intevation/flys/exports/XYChartGenerator.java
--- a/flys-artifacts/src/main/java/de/intevation/flys/exports/XYChartGenerator.java	Wed Dec 05 23:58:26 2012 +0100
+++ b/flys-artifacts/src/main/java/de/intevation/flys/exports/XYChartGenerator.java	Thu Dec 06 13:10:27 2012 +0100
@@ -130,9 +130,10 @@
         public void addArea(StyledAreaSeriesCollection series) {
             this.datasets.add(series);
             List<?> allSeries = series.getSeries();
-            for (int n = 0; n < allSeries.size(); n++) {
-                includeYRange((XYSeries)allSeries.get(n));
-            }
+            /* We do not include the bounds/ranges, if the area includes
+             * points at "infinity"/BIG_DOUBLE_VALUE, the charts extents are
+             * expanded to include these very small/big value.
+             * This is especially used when showing "area above axis". */
         }
 
         /** True if to be rendered as area. */
@@ -469,9 +470,9 @@
             axisDataset.addArea(area);
         }
         else {
-            // TODO only range merging.
+            /* No range merging, for areas extending to infinity this
+             * causes problems. */
         }
-        //TODO range merging.
     }
 
 


More information about the Dive4elements-commits mailing list