added idabot.start_location, start_locations to the documentation and updated...

added idabot.start_location, start_locations to the documentation and updated their comments to avoid confusion

docs/idabot.rst

@@ -60,6 +60,8 @@ IDABot
    .. autoattribute:: current_supply
    .. autoattribute:: max_supply
    .. autoattribute:: current_frame
+   .. autoattribute:: start_location
+   ,, autoattribute:: start_locations
 
 Debug
 -----

python-api-src/library.cpp

@@ -114,8 +114,8 @@ PYBIND11_MODULE(library, m)
 		.def_property_readonly("tech_tree", &IDABot::GetTechTree, "An instance of the class :class:`library.TechTree`")
 		.def_property_readonly("map_tools", &IDABot::Map, "An instance of the class :class:`library.MapTools`")
 		.def_property_readonly("building_placer", &IDABot::GetBuildingPlacer, "An instance of the class :class:`library.BuildingPlacer`")
-		.def_property_readonly("start_location", &IDABot::GetStartLocation, "CCPosition representing the start location")
-		.def_property_readonly("start_locations", &IDABot::GetStartLocations, "CCPosition representing the start locations")
+		.def_property_readonly("start_location", &IDABot::GetStartLocation, "CCPosition representing the start location, note that it is the depot position that is returned.")
+		.def_property_readonly("start_locations", &IDABot::GetStartLocations, "List of CCPositions representing the start locations, note that it is the depot positions and not the center positions")
 		.def_property_readonly("minerals", &IDABot::GetMinerals, "How much minerals we currently have")
 		.def_property_readonly("current_supply", &IDABot::GetCurrentSupply, "How much supply we are currently using")
 		.def_property_readonly("max_supply", &IDABot::GetMaxSupply, "How much supply we can currently use")