Python error getting fabricaiton part hosts

Developers
1

Capture
Capture.PNG1188×831 83 KB

I use a very similar script to get ancillary data. However when I try modify it to get hosted information I can’t get it to read. I’m trying to read the element ID of the pipe fabrication hangers are hosted to.

The API class I’m trying to use: http://www.revitapidocs.com/2018.2/c74f8adf-a227-098c-b58c-a2998560c0d3.htm

The property I’m after (via Revit Lookup):

Capture2
Capture2.PNG1684×674 45.5 KB

I’m not experienced enough to tell if I’m missing something simple, or if what I’m trying to do is impossible.

2

@L-Vettz please post your dyn and rvt files (or a link)

3

Here’s the script and a dropbox link to a test file.

I’m using dynamo 2.0.2

GetHostedInfoTest.dyn (6.2 KB)

4

The GetHostedInfo() method returns a FabricationHostedInfo object, which is not iterable (explaining the error). Instead of writing for n in HostedInformation you should just do HostIDs.append(HostedInformation.HostId)

1 Like
5

That was it. Thank you!

Capture
Capture.PNG814×549 41.3 KB

1 Like
6

Follow up question. Is it possible to make it output a list of elements instead of a list of element ids that are strings?

I know I can just the clockwork node Element.ByID but why not try something more advanced right?

7

It’s actually quite simple. You can use the GetElement() method of your document. For example:

host_id = HostedInformation.HostId
host_element = Doc.GetElement(host_id)
8

Simple if you know what code is yea. I’m learning all this on the fly, it’s mostly still voodoo to me. Dynamo is my introduction to coding. that also worked though, thanks again!

1 Like
9

I am trying to follow this to get the element host of fabrication pipe hanger.
I do not see any difference between my code and your latest screenshot. However I am still getting iteration error at line 31. I am assuming I need to change the else commands to be like the if commands are?

image

10

The logic has only been fixed in the first conditional statement, so if only 1 element is provided (i.e. Cnt == 1), it is still trying to iterate the FabricationHostedInfo object. Instead of being this:

else:
    HostedInformation = Inpt.GetHostedInfo()
    for n in HostedInformation:
        HostIDs.append(n.HostID)

It should be this:

else:
    HostedInformation = Inpt.GetHostedInfo()
    HostIDs.append(HostedInformation.HostId)

Also note that the property name is case sensitive (HostId instead of HostID), so this should be fixed on line 28 as well.

11

I had thought the else section would have to kinda match the if section. The case of the name totally slipped by me.
Thanks

12

image
image.png585×676 28.9 KB

13

Here is a trimmed down version of the above as well (none of the imports are required and nothing has to be wrapped in a transaction as the document is not being modified).

input = UnwrapElement(IN[0])
host_ids = []

# Make input a list if it is not already
if not hasattr(input, '__iter__'):
	input = [input]

for e in input:
	hosted_info = e.GetHostedInfo()
	host_ids.append(hosted_info.HostId)

OUT = host_ids
14

Thanks, that really cleans it up, and no need for for a list.count node since it is not iterating.