Route V 1.0 #5
Closed
Route V 1.0 #5
Conversation
This file contains hidden or 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
updates: - [github.com/astral-sh/ruff-pre-commit: v0.5.0 → v0.8.6](astral-sh/ruff-pre-commit@v0.5.0...v0.8.6) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* bump ruff version in pre-commit * reup ruff formatting * format notebooks * rerun notebooks * update pyproject.toml
* bump ruff version in pre-commit * reup ruff formatting * resolve pysal#475 * update docs
Signed-off-by: Emmanuel Ferdman <[email protected]>
…tion algorithm (pysal#482) * init * notebook * public extraction api * notebook * notebook * change cluster renumbering * formatting and docstrings * reorder imports * ci change * more ci changes * formatting * more formatting * test failures * typo * Update sa3.py * backwards compat * load data within setup_method * lint * better API * fix tests i broke --------- Co-authored-by: Martin Fleischmann <[email protected]>
* bump libpysal requirement to 4.10 * fix imports * fast_hdbscan * test if its sklearn * skip on old sklearn
updates: - [github.com/astral-sh/ruff-pre-commit: v0.11.13 → v0.12.2](astral-sh/ruff-pre-commit@v0.11.13...v0.12.2) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* bump ruff version in pre-commit * reup ruff formatting * bump ruff + pre-commit * hierarchical testin data storage
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
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.
First shippable verison of route module referenced by #4 (defunct) and the initial PR by @ljwolf.
This work was completed during the Google Summer of Code 2025; a NumFOCUS-sponsored project for the Python Spatial Analysis Library (PySAL). See the project blog. This PR contains files changes from upstream, but my contributions can be found in the
spopt/route/directory.The
spopt.routemodule wraps around theroutingpylibrary to leverage street data for use in solving vehicle routing problems using PyVRP.The user can setup a OSRM instance (described in the new notebook
route.py) and then refer to it in the solve function:As well, the user can decide to not pass a router,
which will solve the problem using haversine distances.
Because
spopt.routeleverages the base functions in routingpy which are shared by all of the library's routers (matrixanddirections), users should be able to pass the other API-enabled routers and get the same functionality if their required keywords are passed torouting_kws(e.g. for Valhalla, this would include the keyword"profile" : "auto"). The OSRM service is considered the 'default' use case, but other services should be simple to incorporate.The new notebook
route.pydescribes the confirmed functional routers in more detail.