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

Several edge types in shortestpath() #5711

Closed
dargolith opened this issue Feb 11, 2016 · 2 comments
Closed

Several edge types in shortestpath() #5711

dargolith opened this issue Feb 11, 2016 · 2 comments
Assignees
Milestone

Comments

@dargolith
Copy link

Hi,

It would be nice to specify more than one allowed edge type for shortestpath. It would mean: "Give me the shortest path only using one of these edge classes".

Is there a reason for it not being supported?

I only briefly glanced at the code, but it seems to me like the inner algorithm itself already supports it.

String   edgeType;
String[] edgeTypeParam;

...

ctx.edgeType = null;
if (iParams.length > 3) {
    ctx.edgeType = iParams[3] == null ? null : "" + iParams[3];
}
ctx.edgeTypeParam = new String[] { ctx.edgeType };

Seems like ctx.edgeTypeParam is the one being used later in the code and it is an array already. Would it work by just allowing an array argument for the function?

@lvca lvca added this to the 3.0 milestone Aug 4, 2017
@katherinessmith
Copy link

#7842 requested the same change. Fix is in 3b660af and will be released in 2.2.30.

@luigidellaquila
Copy link
Member

Thank you very much @katherinessmith

I think we can close this

Thanks

Luigi

@luigidellaquila luigidellaquila modified the milestones: 3.0, 3.0.0-RC1 Dec 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants