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.

Sign up for GitHub

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

Jump to bottom

MultiPoint Layer Handling for Traveling Salesman Optimization faulty #209

Closed
koebi opened this issue Dec 20, 2023 · 2 comments · Fixed by #224
Closed

MultiPoint Layer Handling for Traveling Salesman Optimization faulty #209

koebi opened this issue Dec 20, 2023 · 2 comments · Fixed by #224
Assignees
@merydian

Comments

@koebi
Copy link
Collaborator

koebi commented Dec 20, 2023

Here's what I did

Try calculating a route from one point layer using a MultiPoint Layer as an input, checking one of the TSP optimization boxes

Here's what I got

several different errors

We should investigate this and either disallow MultiPoint or fix any issues.
@merydian
Copy link
Collaborator

Is this relevant for the 1.7 release? Is there any advantage to using MultiPoint Layers? I can't think of one.

@koebi
Copy link
Collaborator Author

koebi commented Dec 21, 2023

No, it's not relevant to the release.
I'm not sure - people use QGIS in many different ways, and we should respect that.

@merydian merydian linked a pull request Feb 7, 2024 that will close this issue
fix: RequestsException with multi points layers and TSP option selected in point layer proc #224
Merged
@koebi koebi closed this as completed in #224 Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@merydian merydian

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@koebi @merydian