Mercurial > dive4elements > river
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); } }