geopython / geopython/GeoHealthCheck

Make 1.3.0 default WMS version

Aperta
#455 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
bug
Lingua principale
Python
Stelle
92
Fork
75
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

This issue https://github.com/geopython/GeoHealthCheck/issues/248 and PR https://github.com/geopython/GeoHealthCheck/pull/307 had pretended to change the default WMS version to 1.3.0, but I believe it still needs some improvement, because version 1.1.1 is still hardcoded in some places:

https://github.com/search?q=repo%3Ageopython%2FGeoHealthCheck%201.1.1&type=code

Also, changing from 1.1.1 to 1.3.0 implies some differences, for instance:
- From `SRS` to `CRS`;
- BBOX from `xmin,ymin,xmax,ymax` to `ymin,xmin,ymax,xmax`

I've tried to change locally all default 1.1.1 to 1.3.0, but WMS Drilldown keeps using `version=1.1.1`, and I could not find why.

At this time I think 1.3.0 should be the default for all tests of GHC, and it seems it is not, even after https://github.com/geopython/GeoHealthCheck/pull/307.

**Context (please complete the following information):**
- GeoHealthCheck Version 0.9.0

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia dal codice e dai test restituiti dalla ricerca nel repository per 1.1.1 hardcoded, quindi segui il percorso WMS Drilldown che produce ancora version=1.1.1. Verifica le differenze di WMS 1.3.0 indicate nell’issue, incluse SRS versus CRS e l’ordinamento di BBOX. Il lavoro è completato quando il valore predefinito è 1.3.0 in tutto il codice pertinente e i test passano con quella versione.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
python
Ambito
backend, testing
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
42/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.