DOSull / weaving-space

tools for making tileable weave units that give the appearance of a woven material for multivariate thematic mapping
MIT License
19 stars 0 forks source link

Sort out behaviour of the render() method of TiledMap #10

Closed DOSull closed 9 months ago

DOSull commented 2 years ago

At the moment because TiledMap is a persistent object each time we run render(), whatever the most recent values of its various attributes are set to 'stick' leading to confusing behaviour. For example

# make tiled map
tm.render()
tm.render(use_ellipse = True)
tm.render()

The third call to render() doesn't reproduce the map made by the first call, because the setting change introduced in the second call 'sticks'. This means replacing any configuration attributes not explicitly set in render() with their defaults.

Maybe the current behaviour is preferable, but we need to at least think about what we want.

DOSull commented 9 months ago

Behaviour is fine.