-
Notifications
You must be signed in to change notification settings - Fork 11
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
Map plausibility polygon #505
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
ka-pr
commented
Aug 6, 2024
- Adds a model of "plausibility polygons" intended to heuristically check in the frontend end how many tenure area potentially intersect with a given occurrence geometry
- The calculation is based on average area of a tenure within a plausibility polygon, and intersected tenures count as warning or error values
- The map component issues a warning or rejects occurrence geometries that are projected to intersect with too many tenures
- The check is calculated on finished-drawing, transformation-end, and modify-end events
- Also adds a resultType "hits" geoserver query to the backend to raise an exception when too many tenure areas intersect
- Plausbility geometries are cached
- Adds a json fixture
- Raises DATA_UPLOAD_MAX_MEMORY_SIZE setting from 2.5 to 20 MB
…r an empty feature instead of true/false
…ther than only the first occurrence
…ince those will be needed to calculate an actual area
…d. Added frontend support for multiple plausibility geometries and frontend is now issueing a warning or error based on projected intersected entities
…usibiility geometry applies to
…k_for_geometry field
… logic to the model forms clean method to allow for pasting of geojson directly in the admin panel
…for intersected features
…up the swal, so might be a bit spammy or annoying
…that is not part of the areas purpose, e.g. roads
xzzy
approved these changes
Aug 6, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sensitivity Check Completed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.