changeset 8722:a83d519155ab

(issue1754) Do not base smoothing radius on calculation range Using the calculation parameters for startkm and endkm in the case that the domain axis had the default extend caused weird behavior when zooming and led to too large radius values for most data that only had valid values on a subset of the caluclation range.
author Andre Heinecke <andre.heinecke@intevation.de>
date Tue, 28 Apr 2015 14:22:47 +0200
parents 1083cb887ffb
children 686d8876edf9
files artifacts/src/main/java/org/dive4elements/river/exports/LongitudinalSectionGenerator2.java
diffstat 1 files changed, 6 insertions(+), 8 deletions(-) [+]
line wrap: on
line diff
--- a/artifacts/src/main/java/org/dive4elements/river/exports/LongitudinalSectionGenerator2.java	Mon Apr 27 17:35:28 2015 +0200
+++ b/artifacts/src/main/java/org/dive4elements/river/exports/LongitudinalSectionGenerator2.java	Tue Apr 28 14:22:47 2015 +0200
@@ -80,14 +80,6 @@
             context.putContextValue("endkm", getXBounds(0).getUpper());
             context.putContextValue("bounds_defined", true);
         }
-        else if (getXBounds(0) == null && getDomainAxisRange() == null) {
-            RangeAccess access = new RangeAccess(artifact);
-            if (access.hasFrom() && access.hasTo()) {
-                context.putContextValue("startkm", access.getFrom());
-                context.putContextValue("endkm", access.getTo());
-                context.putContextValue("bounds_defined", true);
-            }
-        }
         else if (getXBounds(0) == null && getDomainAxisRange() != null){
             RangeAccess access = new RangeAccess(artifact);
             if (access.hasFrom() && access.hasTo()) {
@@ -99,6 +91,12 @@
                 context.putContextValue("bounds_defined", true);
             }
         }
+        /*
+        else if (getXBounds(0) == null && getDomainAxisRange() == null) {
+            .. In this case the automatic zoom of the diagram will provide
+            decent values for the zooom calculation.
+        }
+        */
         super.doOut(bundle, theme, visible);
     }
 }

http://dive4elements.wald.intevation.org