Skip to content

Deprecate GeometryCollection in favor of FeatureCollection (e.g. DelayedVector) #71

Open

Description

@soxofaan

see Open-EO/openeo-processes#253:

In VITO backend code we use GeometryCollection as a collection of separate geometries, for which separate aggregations should be calculated for example. In Open-EO/openeo-processes#253 there is discussion to deprecate GeometryCollection usage as much as possible and promote FeatureCollections instead.

In DelayedVector (and other places) we do things the other way around at the moment: when we get a FeatureCollection, we convert it to a GeometryCollection.

Changing this will probably cause some nuisance, e.g. in the interaction between geopyspark driver and in UDFs that depend on this behavior.

Activity

  1. soxofaan commented on Jun 1, 2021

    @soxofaan
    MemberAuthor

    FYI: Open-EO/openeo-processes#253 has been merged, so now it's "official" that GeometryCollections have to be considered to be a single geometry, and that FeatureCollections have to be used to aggregate over separate geometries.

  2. soxofaan commented on Jun 1, 2021

    @soxofaan
    MemberAuthor

    (internal ticket/ref: EP-3893)

  3. soxofaan commented on Jul 12, 2021

    @soxofaan
    MemberAuthor
  4. soxofaan commented on Feb 23, 2022

    @soxofaan
    MemberAuthor

    related to EP-3981 (add vector cube support) too

  5. added 2 commits that reference this issue on Apr 20, 2022
  6. added a commit that references this issue on Mar 14, 2024
  7. soxofaan commented on Jun 7, 2024

    @soxofaan
    MemberAuthor

    related: MultiPolygon should also not be (ab)used as a collection of geometries: #288

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions