Geometry_Engine: Add HashArray method for core geometry types

BHoM / BHoM_Engine

Internal manipulation of the BHoM

GNU Lesser General Public License v3.0

26 stars 13 forks source link

Geometry_Engine: Add HashArray method for core geometry types #3264

Closed pawelbaran closed 7 months ago

pawelbaran commented 7 months ago

Description:

As in the title, HashArray seems to be missing for a few core geometry types:

TransformMatrix
Cartesian

alelom commented 7 months ago

The reason why we didn't add GeometryHash for some geometric types is because we thought it would never get used with them. In particular, we added support for "non-conceptual" types: everything except Vector, TransformMatrix, Cartesian, and probably a few more. Anyways, it can make sense, for completion's sake, to add support for all -- especially now that we have added this new "aggregated" GeometryHash.

pawelbaran commented 7 months ago

Yes, I see immediate use for transforms, so definitely worth adding 👍