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.

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

UF: Adding the vehicle routing problem (VRP) #224

Merged
merged 11 commits into from
Apr 18, 2023

Conversation

alejomonbar
Copy link
Contributor

Unitary Fund

Adding the vehicle routing problem class with its corresponding test.
https://en.wikipedia.org/wiki/Vehicle_routing_problem

Description

Adding the vehicle routing problem combinatorial optimization problem to the set of problems. This problem comes with the function 'classical_optimization' which uses the 'docplex' solver and plot_solution which plots the 'networkx' graph for a given solution.

Checklist

  • I have performed a self-review of my code.
  • I have commented my code and used numpy-style docstrings
  • I have made corresponding updates to the documentation.
  • My changes generate no new warnings
  • I have added/updated tests to make sure bugfix/feature works.
  • New and existing unit tests pass locally with my changes.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

the test is inside 'test_problems'

@Q-lds Q-lds added enhancement New feature or request unitary_fund A feature supported by an UF grant labels Apr 9, 2023
vishal-ph and others added 6 commits April 12, 2023 14:43
I changed the default value of adding all the subtours to -1. I tried with subtours=None, but function vrp.asdict does not save variables with 'None' value.
@vishal-ph vishal-ph merged commit b5ca80f into entropicalabs:dev Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request unitary_fund A feature supported by an UF grant
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants