Resolve confusion between `Symmetry` and `Transform` classes

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

Resolve confusion between `Symmetry` and `Transform` classes #12

Closed DOSull closed 8 months ago

DOSull commented 9 months ago

Recent changes mean we have two classes that encapsulate more or less the same information, only one should be needed. Probably easily resolved, just needs some attention to how the Symmetries and Shape_Matcher classes output their results.

DOSull commented 8 months ago

Resolved by https://github.com/DOSull/weaving-space/commit/17837aa335683143cc9fc33106edd09af30cf33a and https://github.com/DOSull/weaving-space/commit/a4eb685ea4ffa2e8b5b37e7dc340123b867c2de7