-
Notifications
You must be signed in to change notification settings - Fork 410
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
Fix #9620 better error handling of GeoProcessing #9621
Conversation
For sure this exception is not handled. I'm going to review this better when you think that all the exceptions are correctly handled |
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.
It improved a lot. It works well for me.
Please change the text about the source layer, because a source layer as line is anyway accepted.
Question:
I tried an intersection between raoads and streams, I have no results (theoretically I should have some points)
I tried an intersection between roads and roads, it worked.
I think this is a bug/missing feature of the process.
Test passed @MV88 please backport to the stable branch. Thanks |
backport here #9630 |
Description
Error described in the issue now provides a feedback and the app does not crash with specific use case
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (check one with "x", remove the others)
Issue
What is the current behavior?
Fix #9620
Fix #9603
What is the new behavior?
Breaking change
Does this PR introduce a breaking change? (check one with "x", remove the other)
Other useful information