Skip to content
This repository has been archived by the owner on Aug 28, 2021. It is now read-only.

Dictionaries make the sender hang #49

Open
teocomi opened this issue Nov 6, 2018 · 3 comments
Open

Dictionaries make the sender hang #49

teocomi opened this issue Nov 6, 2018 · 3 comments

Comments

@teocomi
Copy link
Member

teocomi commented Nov 6, 2018

Need to either support them or throw a warning.

image

@alvpickmans
Copy link
Collaborator

GH doesn't seem to support them either, returns null though.
@didimitrie any preference?

@didimitrie
Copy link
Member

🤔 either make them a a generic speckle object with the props set to the dict, or null/warning of sorts.

@teocomi
Copy link
Member Author

teocomi commented Nov 7, 2018

@alvpickmans quick fix is to rename the ToSpeckle and ToNative methods in the converter so that Speckle doesn't try to cast it as a SpeckleObject...
Will push something now.

teocomi added a commit that referenced this issue Nov 7, 2018
teocomi added a commit that referenced this issue Nov 8, 2018
* checking for new versions against appveyor's builds

* updated appveyor to use assemblyversioninfo.cs

* included assemblyversioninfo to 2 more projects

* fix for #49

* fixed some specklepopup xaml, added default sever address

* removed auth token from sender

* fixed RenameLayers that was uber buggy

* fixed bug caused when removed apitoken from sender
teocomi added a commit that referenced this issue Nov 9, 2018
* Fixed no Receiver connection upon load, added RunType for better control on Manual execution, fixed IsEnabled on paste button

* removed inotifypropertychanged to use the nodemodel native one

* Added DebugRevit build config

* wip node sender extension, updated to dynamo 2.1

* Dev entitytags - Custom UserData (#36)

* Added UserData through Geometry Tags

* Fixed weird behaviour with references not being recognized"

* #32 Added UserData with Icons, an updated Speckle to have two categories: I/O and userData

* Handling nested dictionaries structure

* Simplifying but saving the actual DesignScript.BuiltIn.Dictionary

* Added methods to convert DesignScript.BuiltIn.Dictionary

* Update Readme with UserData

* Update Readme.md

* updated version number

* Fixes UserData exception with Meshes (#45)

* Dev (#40)

* Fixed no Receiver connection upon load, added RunType for better control on Manual execution, fixed IsEnabled on paste button

* removed inotifypropertychanged to use the nodemodel native one

* Added DebugRevit build config

* Dev entitytags - Custom UserData (#36)

* Added UserData through Geometry Tags

* Fixed weird behaviour with references not being recognized"

* #32 Added UserData with Icons, an updated Speckle to have two categories: I/O and userData

* Handling nested dictionaries structure

* Simplifying but saving the actual DesignScript.BuiltIn.Dictionary

* Added methods to convert DesignScript.BuiltIn.Dictionary

* Update Readme with UserData

* Update Readme.md

* updated version number

* fixing #44

* and fixing #44 again, as it seems I'm terrible at merging branches.

* update core

* updated speckle core

* updated specklecore

* updated speckle core

* swapped back to 2.0

* Alv/fix polycurves (#50)

* Fix #48 - no planar closed polylines convert to open polycurves.

* Disposing polycurve after closing it as it's not returned.

* checking for new versions against appveyor's builds

* updated appveyor to use assemblyversioninfo.cs

* included assemblyversioninfo to 2 more projects

* fix for #49

* fixed some specklepopup xaml, added default sever address

* removed auth token from sender

* fixed RenameLayers that was uber buggy

* fixed bug caused when removed apitoken from sender

* Matteo/dev (#51)

* checking for new versions against appveyor's builds

* updated appveyor to use assemblyversioninfo.cs

* included assemblyversioninfo to 2 more projects

* fix for #49

* fixed some specklepopup xaml, added default sever address

* removed auth token from sender

* fixed RenameLayers that was uber buggy

* fixed bug caused when removed apitoken from sender

* Added node manager win

* release 0.0.6
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants