funky todo in PairwiseShortestPath

coli-saar / alto

Alto, the Algebraic Language Toolkit

Other

17 stars 2 forks source link

funky todo in PairwiseShortestPath #48

Open jgroschwitz opened 5 years ago

jgroschwitz commented 5 years ago

there's an odd bug in the PairwiseShortestPath class where DirectedMultigraph#containsEdge sometimes seems to ignore directionality. current workaround works, but might be worth looking into

alexanderkoller commented 5 years ago

We should upgrade to a current version of jgrapht at our earliest opportunity. Chances are that this will fix the bug.

akoehn commented 5 years ago

Iirc, the new version has a different api. I upgraded all libraries to the latest compatible version when switching alto to gradle.

jgroschwitz commented 5 years ago

I just made the jgrapht update a separate issue (#51), since I also encountered it elsewhere. Will try to do this soon.