Resolve "Test validating messages with references to GeoJson schema in both JS and python"
- No files found.
added 2 commits
added 1 commit
- a00c80f2 - refactor: load geojson defs from published schemas
added 2 commits
GeoJSON Point accepts a minimum of 2 coordinates but sets no maximum.
If we use GeoJSON Point to specify lat/long do we want to include altitude/depth and if so do we just have a convention that says [x(long), y(lat), z(depth), a(?)(altitude)] so you can provide points as 2D, 3D, or 4D as required?
Alternatively do we just use GeoJSON Point for lat/long and keep depth/altitude as separate named parameters? Or do we have a 3D Point [x,y,z] with a parameter that says whether z refers to depth or altitude?
added 20 commits
Toggle commit listadded 8 commits
-
b262672b - 1 commit from branch
67-mas-dt
- 2443ec3a - refactor: load geojson defs from published schemas
- a3a933b4 - chore: add flake8 config
- 10735293 - lint: fix black and flake8 errors
- 5e0ee51a - refactor: change geo data to geojson refs
- 2316c1dd - feat: import geojson multi definitions
- 28a71b73 - lint: run black
- d26ee222 - chore: recompile schema
Toggle commit list-
b262672b - 1 commit from branch
Hmm - Importing the individual geographies works OK but for some reason importing the
Feature
orFeatureCollection
schemas fails and I can't immediately see why.We can probably do without
FeatureCollection
but I think we needFeature
for the primitive list because we want primitives to accept different geometry types.So far I've removed
$id
and$schema
from the published schemasI think removing properties that don't alter the structure or validation of the data is probably fine. I might be able to make
Feature
work with a bit more of that.If I have to do anything else to make
Feature
work then I think it needs a different answer.added 18 commits
-
d26ee222...43ffbbbf - 7 commits from branch
67-mas-dt
- 3daab322 - refactor: load geojson defs from published schemas
- 673cab5c - chore: add flake8 config
- bbded930 - lint: fix black and flake8 errors
- 72554c78 - refactor: change geo data to geojson refs
- 3bdc5ca4 - feat: import geojson multi definitions
- 591aa906 - lint: run black
- 735243de - chore: recompile schema
- dc538669 - refactor: revert change to emergency schema
- c307438b - refactor: make imported schemas available to tests
- 8e9a8b4c - fix: correct argparse default behaviour
- 84b9750e - feat: downgrade geojson to 3.0.x compatible schema
Toggle commit list-
d26ee222...43ffbbbf - 7 commits from branch
added 1 commit
- f1b4198c - refactor: replace deprecated validator interface
approved this merge request
- You're only seeing other activity in the feed. To add a comment, switch to one of the following options.
- latest versiondcbfd70c13 commits,
- version 11f1b4198c12 commits,
- version 1084b9750e11 commits,
- version 9d26ee2228 commits,
- version 8d26ee2227 commits,
- version 7d7e8e1046 commits,
- version 6d7e8e1046 commits,
- version 56c50fcfa5 commits,
- version 489a1738b3 commits,
- version 389a1738b3 commits,
- version 2a00c80f21 commit,
- version 1c3c8df982 commits,