Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adresseerbaarobject/pand bbox param voldoet niet de definitie van 'OGC API Features specificatie - bbox definitie' zoals aangegeven #544

Open
KayodeBakker opened this issue Apr 20, 2022 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@KayodeBakker
Copy link
Collaborator

Beschrijf de bug
De adresseerbaarobject en pand bevragingsmogelijkheden bieden de optie aan voor bbox waar in de documentatie vervolgens staat: "Voor een definitie van bbox, zie OGC API Features specificatie - bbox definitie". Deze definitie dicteert een standaard voor bbox gebruik en geeft helder aan:

Lower left corner, WGS 84 longitude

Lower left corner, WGS 84 latitude

Upper right corner, WGS 84 longitude

Upper right corner, WGS 84 latitude

Ofwel min-x, min-y, max-x, max-y maar de API dicteert dat er min-x, max-x, min-y, max-y wordt gebruikt en daarmee niet aan de OGC standaard voldoet. Dit is apart aangezien GIS tools ook gebruik maken van de OGC standaard en ik kan me niet voorstellen dat dit een bewuste keuze was.

To Reproduce
Stappen om de bug te reproduceren

  1. Ga naar https://api.bag.acceptatie.kadaster.nl/esd/huidigebevragingen/v1/adresseerbareobjecten?bbox=118314,118326,404843,404850
  2. Zie error.
{
    "type": "https://tools.ietf.org/html/rfc7231#section-6.5.1",
    "title": "Een of meerdere parameters zijn niet correct.",
    "status": 400,
    "detail": "Bad request.",
    "instance": "https://api.bag.acceptatie.kadaster.nl/esd/huidigebevragingen/v1/adresseerbareobjecten?bbox=118314,118326,404843,404850",
    "code": "paramsValidation",
    "invalidParams": [
        {
            "name": "bbox",
            "code": "geometryMismatch",
            "reason": "Waarde is niet conform opgegeven CRS."
        }
    ]
}
  1. Ga naar https://api.bag.acceptatie.kadaster.nl/esd/huidigebevragingen/v1/adresseerbareobjecten?bbox=118314,404843,118326,404850
  2. Krijg normaal resultaat.

Verwacht gedrag
De bbox param dient te worden geïmplementeerd zoals in de OGC standaard gespecificeerd staat ofwel 'min-x, min-y, max-x, max-y' zoals in de API documentatie ook geschreven staat als ondersteunde standaard.

@strijm
Copy link
Collaborator

strijm commented Apr 20, 2022

@KayodeBakker
Je schrijft: "de API dicteert dat er min-x, max-x, min-y, max-y wordt gebruikt". Ik zie dat niet in de API specificatie of feature beschrijving staan en volgens mij heb je met jouw test uitstekend aangetoond, dat de API reageert zoals beschreven in de OGC API Features en dat min-x, min-y, max-x, max-y (stap 3) wordt geaccepteerd en min-x, max-x, min-y, max-y (stap 1) niet. Of zie ik iets over het hoofd?

@Processfive
Copy link

De huidige implementatie is verkeerd (min-x, max-x, min-y, max-y) en voldoet niet aan de OGC definitie (min-x, min-y, max-x, max-y. Zie het als twee coördinaten, linksonder (min-x, min-y) en rechtsboven (max-x, max-y).

@strijm
Copy link
Collaborator

strijm commented Apr 26, 2022

Als het volgende wordt opgegeven:
bbox=118314,404843,118326,404850
dan zijn dat de waarden: min-x, min-y, max-x, max-y
oftewel de coördinaten linksonder en rechtsboven van een bbox.
De coördinaten (118314,404843) en (118326,404850) kun je opzoeken in een (RD) viewer, bv. BAG of PDOK.
Dit is stap 3 in Kayo's test. Dit is correct en conform OGC API Features.

Als het volgende wordt opgegeven:
bbox=118314,118326,404843,404850
dan zijn dat de waarden min-x, max-x, min-y, max-y
oftewel de coördinaten staan niet in de juiste volgorde voor een bbox.
De coördinaten (118314,118326) en (404843,404850) kun je niet opzoeken in een (RD) viewer, bv. BAG of PDOK omdat ze buiten het RD stelsel vallen.
Dit is stap 1 in Kayo's test. Dit is niet correct en niet conform OGC API Features.

M.i. is de werking van de API dus conform OGC API Features.

@Processfive
Copy link

Kayo heeft helaas de requests door elkaar gehaald. Jullie huidige implementatie is min-x, max-x, min-y, max-y en dat is niet correct.

@fsamwel
Copy link
Collaborator

fsamwel commented Apr 26, 2022

@KayodeBakker @Processfive ik begrijp niet zo goed hoe jullie tot de conclusie komen dat de implementatie min-x, max-x, min-y, max-y vereist.
Bijvoorbeeld ik kan /adresseerbareobjecten?bbox=135228,457502,135246,457457 vragen en dan krijg ik objecten terug.
En /adresseerbareobjecten?bbox=135228,135246,457502,457457 geeft een geometryMismatch
Dus zover ik kan zien accepteert de API juist min-x, min-y, max-x, max-y wel en geeft het een fout op min-x, max-x, min-y, max-y.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

8 participants