Infiziert90 / SubmarineTracker

Track and Build
MIT License
8 stars 7 forks source link

Identical total exp but optimizer picked slower route #10

Closed madness800 closed 11 months ago

madness800 commented 12 months ago

Hi,

Not sure if it's a bug or intended but the optimized route picked the slower route despite the total exp yield is identical on both route.

This is the mogship link to cross reference the information https://www.mogship.com/submarines/exp-calculator?rank=97&slot0=23&slot1=24&slot2=25&slot3=10&excludeSectors=50,84,85,86,87,88,89,90,91,92,93&excludeMaps=5

Suggested route & duration image

Faster time by a few mins image

Many thanks and keep up with the good work!

Infiziert90 commented 12 months ago

Not sure if it's a bug or intended but the optimized route picked the slower route despite the total exp yield is identical on both route

The plugin picks routes based on the distance first, so if both have 185, it just picks one of the two After it picked the lowest distance route the duration calculation is done Maybe i will fix it in the future but the time save is so minimal that it prolly isn't even worth it

madness800 commented 12 months ago

Yeah, like you said it is minimal if it was anything outrages then it probably needs to be addressing. I figured to report it so you're aware, since I'm not sure if it was bug, intentional or an oversight.

However, the dev inside of me screams out I should make an issue on it. xD

Infiziert90 commented 11 months ago

image Sorts it correctly now, 42:08:11