search

IITC-CE / ingress-intel-total-conversion

intel.ingress.com total conversion user script with some new features. Should allow easier extension of the intel map.
https://iitc.app
ISC License
288 stars 110 forks source link

"fly links", "tidy links" and already existing links #202

Open modos189 opened 5 years ago

modos189 commented 5 years ago

I was sent a proposal to change the way the plugins "fly links" ans "tidy links" work.

My suggestion is to take into account already existing links, i.e. not to suggest links that cross other links.

I'd like that change, too.

johnd0e commented 5 years ago

Tidy links old issue: https://github.com/iitc-project/ingress-intel-total-conversion/issues/719 Fly links seems to have better chances, but you better ask it's author here: https://github.com/iitc-project/ingress-intel-total-conversion/pull/1084

johnd0e commented 5 years ago

i.e. not to suggest links that cross other links.

It is only 1 of possible options. E.g. someone may prefer to take into account friend links only, and ignore enemy's. Or consider only visible links (as we can hide unrelated layers). Take into account drawn items could be also useful.

johnd0e commented 5 years ago

As for implementation in tidy-links.

Already said that delaunay triangulation is solid algorithm, and we cannot 'just add our minor refinement' there. So here completely another algorithm required, perhaps custom. It is definitely possible, but we need someone with strong mathematical background to implement it.