jira.codehaus.org

  • Log In Access more options
    • Online Help
    • Keyboard Shortcuts
    • About JIRA
    • JIRA Credits
    • What?s New
  • Dashboards Access more options (Alt+d)
  • Projects Access more options (Alt+p)
  • Issues Access more options (Alt+i)
  • GeoTools
  • GEOT-1724

GeneralEnvelope should accepts envelopes crossing the antimeridian

  • Log In
  • Views
    • XML
    • Word
    • Printable

Details

  • Type: Improvement Improvement
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 2.5-M0
  • Fix Version/s: 2.7.5
  • Component/s: referencing
  • Labels:
    None

Description

GeneralEnvelope.setRange(int, double, double) currently contains this code:

if (minimum > maximum) {
    // Make an empty envelope (min == max)
    // while keeping it legal (min <= max).
    minimum = maximum = 0.5 * (minimum + maximum);
}

This approach has two problems:

  • It is a cause of bug that may be long to identify, since the coordinates are silently modified. An exception, or at the very least a warning could save developper time.
  • WCS authorizes (with special treatment) cases where maximum < minimum at least in the longitude case. They are envelope crossing the antimeridian.

We should accept maximum < minimum only for axis declared with RangeMeaning.WRAPAROUND. For other axis, we need to decide if we should throws an exception, reorder the minimum and maximum or just log a warning.

Every GeneralEnvelope methods like contains will need to be updated for taking the antimeridian crossing case in account.

Issue Links

depends upon

Task - A task that needs to be done. GEOT-2117 Review GeoToolkit fixes

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Open - The issue is open and ready for the assignee to start work on it.
is related to

Improvement - An improvement or enhancement to an existing feature or task. GEOT-1745 MapProjection: relax coordinate checks/make them configurable

  • Major - Major loss of function.
  • Reopened - This issue was once resolved, but the resolution was deemed incorrect. From here issues are either marked assigned or resolved.

Activity

  • All
  • Comments
  • History
  • Activity
Hide
Permalink
Martin Desruisseaux added a comment - 02/Dec/10 5:30 AM
See also http://jira.geotoolkit.org/browse/GEOTK-130
Show
Martin Desruisseaux added a comment - 02/Dec/10 5:30 AM See also http://jira.geotoolkit.org/browse/GEOTK-130

People

  • Assignee:
    Unassigned
    Reporter:
    Martin Desruisseaux
Vote (0)
Watch (0)

Dates

  • Created:
    06/Mar/08 8:29 AM
    Updated:
    21/Dec/11 10:38 AM
  • Atlassian JIRA (v5.0.4#731-sha1:3aa7374)
  • Report a problem
  • Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.