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

HoffmanPavleyRankedShortestPathAlgorithm.ToString() #174

Open
jnyrup opened this issue Jan 31, 2018 · 1 comment
Open

HoffmanPavleyRankedShortestPathAlgorithm.ToString() #174

jnyrup opened this issue Jan 31, 2018 · 1 comment

Comments

@jnyrup
Copy link
Contributor

jnyrup commented Jan 31, 2018

The ToString in HoffmanPavleyRankedShortestPathAlgorithm is implemented as

public override string ToString()
{
    return String.Format("{0} at {1} {2}", this.Weight, this.DeviationEdge);
}

This will obviously throw an exception as the number of arguments doesn't match the expected number in the format string.

As I'm not sure what the intended format is, I haven't submitted a PR to fix this.

@KeRNeLith
Copy link

In my fork I moved to string interpolation keeping the 3 args in a way similar to debugguer display (see here).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants