Currently the Topology class is only loosely linked to the rest of the package. That may be completely OK.
But... some thought should be given to the integration. Perhaps all TileUnit objects should instantiate a Topology and store it as an attribute. Perhaps they only need to do that if changes which require a Topology are being carried out.
WeaveUnits with spacing < 1.0 can't sustain a Topology, so this also needs to be considered.
Currently the
Topologyclass is only loosely linked to the rest of the package. That may be completely OK.But... some thought should be given to the integration. Perhaps all
TileUnitobjects should instantiate aTopologyand store it as an attribute. Perhaps they only need to do that if changes which require aTopologyare being carried out.WeaveUnits withspacing < 1.0can't sustain aTopology, so this also needs to be considered.