search

speckleworks / SpeckleCore

Check a brand new Speckle at: https://github.com/specklesystems
https://speckle.systems
MIT License
38 stars 17 forks source link

SpecklePoints[] instead of double[]? #65

Closed Eckart-S closed 6 years ago

Eckart-S commented 6 years ago

Wouldn't it make sense to use SpecklePoints instead of double[] Points { get; set; } for Curves etc.? Name "Points" might be misleading as it's in reality Point coordinates and not SpecklePoints.

didimitrie commented 6 years ago

it would be good dev-experience wise, but what if i would add a default extension for arrays that you would just call .ToPoints on an array? Points[] fromArray( array ) ?

I'm insisting on this because once you make a speckle object, in theory it should get a hash, _id, etc. that will increase a lot the transport size

didimitrie commented 6 years ago

i'm going to close this as it's not happening in 1.x.x