Skip to content

Traveling Salesman Problem Added #5370

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

Closed
wants to merge 1 commit into from
Closed

Traveling Salesman Problem Added #5370

wants to merge 1 commit into from

Conversation

DBasu2610
Copy link

This is the Traveling Salesman Problem which gives the shortest possible route among a set of cities where every city is visited atleast once and the starting city is returned to

It is done by using the Held-Karp algo, with memorization

This approach is much more efficient than the brute-force approach for small to moderately sized instances of TSP.

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 0% with 19 lines in your changes missing coverage. Please review.

Project coverage is 45.82%. Comparing base (56f97c4) to head (3bae82c).

Files Patch % Lines
...hms/datastructures/graphs/TravelingSalesmanDP.java 0.00% 19 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5370      +/-   ##
============================================
- Coverage     45.89%   45.82%   -0.07%     
  Complexity     2857     2857              
============================================
  Files           525      526       +1     
  Lines         15418    15437      +19     
  Branches       2917     2924       +7     
============================================
- Hits           7076     7074       -2     
- Misses         8047     8067      +20     
- Partials        295      296       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@alxkm
Copy link
Contributor

alxkm commented Aug 23, 2024

Hi @DBasu2610 , build job does not pass. It has some checkstyle issues. Please fix them and we can start the review

@DBasu2610 DBasu2610 closed this Aug 23, 2024
@DBasu2610 DBasu2610 reopened this Aug 23, 2024
@DBasu2610 DBasu2610 closed this Aug 23, 2024
@DBasu2610 DBasu2610 reopened this Aug 23, 2024
@DBasu2610 DBasu2610 closed this Aug 23, 2024
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.

3 participants