'Get surrounding elements' node detects more elements than necessary

[joris.vanbossche]

Can someone tell me what’s wrong with my script or with my project?
I’m trying to send information from my rooms to the ceilings, but for an unknown reason the ‘getsurroundingelements’ node gives me more rooms than I expected:

Is there something wrong with the script or is there something not correct in my model?
Files: https://www.dropbox.com/sh/akfpkk1utjctuoz/AABcMR53rBOBQgsrdOdfGEYea?dl=0

[Yna_Db]

Hi, Get Surrounding Elements seems to use a bounding box method:

Element.Room node (Rhythm) package - null values

Hi guys! I had the chance to finish this script. I found that my best (and accurate) solution was to use Rooms Boundrais to create dynamo extrusions, then increase their thickness, and then Element.BoundingBox for the windows/curtain panels, then using GeometryDoesItersect I could have a mask to use it later to have curtain panels/windows grouped by rooms. Also I found that the node "Get surrounding elements" was using the BoundingBox method. that's why it was not giving me accurate result…

[john_pierson]

There;s also a good method of using Raybounce to grab ceilings discussed here.

Ceiling height to room parameter

I’m following the script that is explained here: http://dynamobim.org/forums/topic/ceiling-height-in-a-room-parameter/ As result I have a ceiling height in the room parameter ‘comments’, but it’s not the correct one. How can I change my script so that the ceiling height is filled in in the correct corresponding room parameter? hoogte vals plafond naar ruimte parameter.dyn (19.3 KB)

[joris.vanbossche]

@Yna_Db, does this means that the Get Surrounding Elements doesn’t work for non-rectangular rooms?

@john_pierson, I’ve tried to modify your script so that it works for all rooms in my project, without selecting them, but than I get following error:

[john_pierson]

Are all of the rooms reporting a location? You may need to filter out unplaced rooms first.