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

Support reprojection #432

Merged
merged 19 commits into from
Jan 16, 2025
Merged

Support reprojection #432

merged 19 commits into from
Jan 16, 2025

Conversation

margrietpalm
Copy link
Contributor

This PR collects all previous PRs related to this epic, no review needed.

margrietpalm and others added 19 commits December 2, 2024 13:38
…di-modelchecker into margriet_schema_300_leftovers
* Collect all present ForeignKeyChecks and modify generate_foreign_key_checks to generate these checks

* Move all foreign key checks in QueryChecks to actual ForeignKeyChecks

* Add unique check for boundary_condition_1d.connection_node_id
Checks for

* Formatting of dry_weather_flow.distribution string

* Number of values in dry_weather_flow.distribution (should be 24)

* Sum of values in dry_weather_flow.distribution (should be 100)
* Check if geometries for orifice, weir and pipe match their connection nodes

* Extend LinestringLocationCheck for all checks for linestring geometries that should match objects

* Replace CrossSectionLocationCheck with generic PointLocationCheck
…di-modelchecker into margriet_schema_300_leftovers
* Change CRS in all tests to 28892

* Disable RasterHasMatchingEPSGCheck (should be fixed in ticket 414)

* Remove geo_query and use explicit queries instead now transformations are no longer needed
* Add functionality to find CRS for model based on schematisation and dem

* Add checks to ensure model CRS is valid, projected and in metres

* Add EPSGGeomCheck and RasterHasMatchingEPSGCheck to check if geometry and raster epsg match model epsg

* Add factories for EPSGGeomCheck and RasterHasMatchingEPSGCheck
@margrietpalm margrietpalm merged commit 0f96faf into master Jan 16, 2025
7 checks passed
@margrietpalm margrietpalm deleted the margriet_implicit_crs branch January 16, 2025 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant