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

Dictionary issue #3100

Closed
andydandy74 opened this issue Nov 15, 2014 · 6 comments
Closed

Dictionary issue #3100

andydandy74 opened this issue Nov 15, 2014 · 6 comments
Assignees
Labels
bug needs more info Needs more information from the reporter. tracked

Comments

@andydandy74
Copy link
Contributor

Has something changed regarding the way dictionaries are treated compared to 0.6.3?
I am trying to migrate some Python code to 0.7 which - since there's no interaction whatsoever with the Revit API - in theory should not need any massaging at all. Instead I get this warning:

Warning: IronPythonEvaluator.EvaluateIronPythonScript operation failed. 
'string' does not contain a definition for 'Key'

@holyjewsus - I know you've been working with Python dictionaries as well. Any thoughts?
File is here:
https://github.com/CAAD-RWTH/ClockworkForDynamo/blob/master/issues/3100_Python_dictionaries.dyn

@mjkkirschner
Copy link
Member

tracked internal as 5108, this is a known issue, at this time my suggestion is to wrap up the dictionary inside of another object with a dict field, if you need to move dictionaries around the graph, then unwrap it in another node.

You can also do this with CBNS: which might be useful...

screen shot 2014-11-15 at 6 29 37 pm

@ikeough
Copy link
Contributor

ikeough commented May 1, 2016

@mjkkirschner still an issue?

@arif-hanif
Copy link

this is still an issue

@epeter-
Copy link

epeter- commented Sep 14, 2017

@mjkkirschner @ikeough soon we will have a 3rd anniversary with that issue

@adambear82
Copy link

adambear82 commented Aug 16, 2018

this is stopping me from using attrib in xml.etree.ElementTree 😞

@johnpierson johnpierson added needs investigation Needs looked into further. and removed Python labels Oct 4, 2018
@johnpierson johnpierson added tracked needs more info Needs more information from the reporter. and removed needs investigation Needs looked into further. labels Dec 14, 2018
@johnpierson
Copy link
Member

I believe this has been addressed in this PR #9296

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug needs more info Needs more information from the reporter. tracked
Projects
None yet
Development

No branches or pull requests

7 participants