Bug with CatalystLE: unable to find all base locations

[davechurchill]

Bug with CatalystLE: unable to find all base locations

[Archiatrus]

This is most likely due to UnitType::isMineral() is not including all types of minerals/gases. CommandCenter can not find the base locations since it can not find any minerals on the map. See here my fixed isMineral().

I can make a pull request for this, if it is wanted.

I have this problem on each other new maps of ladder 2018 too.

[davechurchill]

Thanks for letting me know. The base location calculation relies on a distance for clustering resources, this can be tweaked here:

https://github.com/davechurchill/commandcenter/blob/master/src/BaseLocationManager.cpp#L20

However right now it's using air distance instead of ground distance, so making it too big will cause some locations on some maps to include minerals that it shouldn't. I can't remember why I don't currently use ground distance, probably because it was a little slow at the time of writing.

If you play around with that number (currently 12 build tiles) you can get better results on some maps

[janemrich]

Thank you for this information. Fortunatly the other maps are enough for my needs