search

Geonovum / KP-APIs

26 stars 40 forks source link

ontwikkelbranch voor definitieve versie geo module #598

Closed lvdbrink closed 1 year ago

lvdbrink commented 1 year ago

Commentaar uit #597 verwerken.

In #597 staat nu nog 1 open punt. Als duidelijk is wat we daar mee doen kan ik dit ook verwerken en kan dit PR gereviewd/gemerged worden.

Inmiddels opgelost.

closes #597

lvdbrink commented 1 year ago

Open punt in #597 nu ook opgelost. @strijm @PB-GNM graag jullie goedkeuring!

lvdbrink commented 1 year ago

Typo gecorrigeerd, zin over geometrietypen geherformuleerd en de suggestie van Pieter voor de note overgenomen. Graag nog 1x review!

strijm commented 1 year ago

@lvdbrink ik heb Lennard gevraagd even mee te kijken naar de aanpassing omtrent lange lijnen.

strijm commented 1 year ago

Input van Lennard:

1.

het is niet zo dat het probleem alleen optreedt als het CRS voor het filter anders is dan het CRS van de data. Het probleem treedt ook op bij gebruik van 1 CRS. Het is wel zo dat het effect afhankelijk is van het CRS waarin wordt gefilterd. Bij CRS84 treedt het probleem eerder op dan bij RD.

Dit komt overeen met https://github.com/Geonovum/KP-APIs/issues/597#issuecomment-1538107194.

Suggestie voor aanpassing:

Spatial operations like intersects and within in combination with a filter geometry (e.g. bbox) or resource geometry containing long lines, may result in erroneous responses since a straight line between two coordinates in a CRS is, depending on the CRS, not a straight line in reality.

2.

The default CRS for GeoJSON and for OGC API Features is CRS84 (OGC:CRS84), this is WGS 84 with coordinate order longitude-latitude.

Dit is onjuist. Suggestie voor aanpassing:

The default CRS for GeoJSON and for OGC API Features is CRS84 (OGC:CRS84), this CRS uses the WGS 84 datum with an ellipsoidal coordinate system in the order longitude-latitude.

3.

New ensemble member CRSs may be released in future (e.g. ITRF2020 has been released as a realization for ITRF), these new realizations shall be used instead of older realizations (e.g. ITRF2014).

Suggestie voor aanpassing:

New ensemble member CRSs may be released in future (e.g. ITRF2020 has been released as a realization for ITRF). These new realizations shall be used instead of older realizations in case of ITRF. In case of ETRF however, National Mapping Agencies have agreed on using ETRF2000 instead of newer realisations (e.g. ETRF2014).

4.

Bij CRS84 en CRS84h in de CRS tabel staat: (φ, λ) en (φ, λ, h) maar dat moet (λ, φ) en (λ, φ, h) zijn.

PB-GNM commented 1 year ago

Ik ben het helemaal eens met de 4 verbeteringen uit de input van lennard.

lvdbrink commented 1 year ago

@strijm ik heb net de suggesties van jou/Lennard verwerkt. Wil je nog één keer kijken?