Skip to content

Edge demand#4

Draft
FormelessPuppy41 wants to merge 23 commits into
mainfrom
edge_demand
Draft

Edge demand#4
FormelessPuppy41 wants to merge 23 commits into
mainfrom
edge_demand

Conversation

@FormelessPuppy41
Copy link
Copy Markdown
Owner

This PR:

  • Closes #xxxx.
  • Adds and passes relevant tests.
  • Has well-formatted code and documentation.
Details

Notes:

Please read our contributing guidelines first.
In particular:

  • You must add tests when making code changes.
    This keeps the code coverage level up, and helps ensure the changes work as intended.
  • When fixing a bug, you must add a test that would produce the bug in the master branch, and then show that it is fixed with the new code.
  • New code additions must be well formatted. Changes should pass the pre-commit workflow, which you can set up locally using pre-commit.
  • Docstring additions must render correctly, including escapes and LaTeX.
  • Finally, it is essential that all contributions in this PR are license-compatible with PyVRP's MIT license.
    Please check that this PR can be included into PyVRP under the MIT license.

N-Wouda and others added 23 commits March 31, 2026 13:35
…yVRP#1090)

* Add failing test

* Fix evaluation of moves that leave a route empty (without clients)

* Remove Proposal::size()

* Implement numClients() for segments
* Bump configure-pages and deploy-pages actions to versions that use Node 24

* Also bump codecov

(codecov@v5 depends on github-script@v7, which runs with Node 20. github-script@v8 runs with Node 24, and codecov@v6 has that dependency)
* Split off data components to separate header and source files

* Split tests
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.

2 participants