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

Mising method call error #18

Closed
Bogdaan opened this issue May 22, 2015 · 1 comment · Fixed by #36
Closed

Mising method call error #18

Bogdaan opened this issue May 22, 2015 · 1 comment · Fixed by #36
Labels
bug easy pick
Milestone
v0.8.2

Comments

@Bogdaan
Copy link

Bogdaan commented May 22, 2015

Fatal error: Call to undefined method Fhaculty\Graph\Graph::getWeight() in /vendor/graphp/algorithms/src/TravelingSalesmanProblem/Bruteforce.php on line 88
@clue clue added the bug label Jun 6, 2015
@clue
Copy link
Member

clue commented Jun 6, 2015

Thanks for spotting and reporting!

This error will (only) appear if you invoke the (undocumented, but public) Bruteforce::setUpperLimitMst() method.

This ought to be fixed as part of #10. Until then, I'm also accepting PRs that fix this issue in the meantime. Any input is welcome!

@clue clue added this to the v0.9.0 milestone Oct 9, 2018
@clue clue added the easy pick label Oct 10, 2018
@clue clue mentioned this issue Oct 13, 2018
Merged
@clue clue closed this as completed in #36 Oct 13, 2018
@clue clue modified the milestones: v0.9.0, v0.8.1, v0.8.2 Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug easy pick
Projects
None yet
Milestone
v0.8.2
Development

Successfully merging a pull request may close this issue.

Fix setting upper limit for TSP bruteforce via MST algorithm clue-labs/algorithms
2 participants
@clue @Bogdaan