SonOfTheStars / Logistics-Pipes-2

MIT License
30 stars 7 forks source link

Added support for using Dijkstras Algorithm #8

Closed Borimino closed 7 years ago

Borimino commented 7 years ago

Added a NetworkSimplifier, which simplifies the list of nodes in a network to a list of junctions and destinations in a network. Also added DijkstraRouter, which uses Dijkstras algorithm to search through the simplified network. I am not fully able to get a route out of DijkstraRouter yet though.

Once routing has been completely implemented, I believe that this should make routing faster, at the expense of doing some computing when a new node is added to the network.

SonOfTheStars commented 7 years ago

Hey, First off all you had me really suprised there that someone is actually working with my crappy codebase 😛 I very much apprechiate that effort of yours! I'm desperate for help as my time to work on this is very limited.

Id definitly love to test your implementation. Yet that requires 2 Things:

Ill write up some more detail shortly.

First of all was this approach of yours Run as a test yet? Was it able to find the Route?

SonOfTheStars commented 7 years ago

@Borimino Please, feel free to contact me on Discord about this. My server is available here https://discord.gg/SSr23qH

Borimino commented 7 years ago

It now works in some networks, though I have also found at least one configuration, where it results in an infinite loop when trying to resolve the path gotten from Dijkstras algorithm