Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify cast from IGH_Goo to GH_Goo<> #339

Open
cdriesler opened this issue Jul 8, 2023 · 0 comments
Open

Simplify cast from IGH_Goo to GH_Goo<> #339

cdriesler opened this issue Jul 8, 2023 · 0 comments
Labels
gh Grasshopper 1 help wanted Extra attention is needed @nodepen/converters NodePenConvert de/serialization

Comments

@cdriesler
Copy link
Member

The current version of the NodePen Speckle converter has a wonderfully simple overload for ConvertToSpeckle that takes a GH_Goo<T> of any kind and converts it to the Speckle geometry that we need.

Unfortunately, the thing that would like to call it only has type-safe access to an IGH_Goo object. So close! And yet, not close enough to invoke the simple generic convert method.

As a result, I had to write this beauty as a little bridge. There has to be a better, terse, and more generic way to do this!

@cdriesler cdriesler added help wanted Extra attention is needed @nodepen/converters NodePenConvert de/serialization .NET Pull requests that update .net code gh Grasshopper 1 and removed .NET Pull requests that update .net code labels Jul 8, 2023
@cdriesler cdriesler pinned this issue Jul 9, 2023
@cdriesler cdriesler unpinned this issue Jul 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gh Grasshopper 1 help wanted Extra attention is needed @nodepen/converters NodePenConvert de/serialization
Projects
Status: No status
Development

No branches or pull requests

1 participant