At the end of the definition, just change the name of the parameter that is feeding into the “Element.SetParameterByName” node. Just now it is writing to the “Comments” parameter because that is what it is being told to do.
The problem is not the parameter, the problem is the value. The value that is filled in is not the one of the corresponding ceiling height but of a random ceiling of another room. There is so a mismatch between the ceiling height and the rooms…
The node seems to work for me as expected. I am only using it on a small project with 10 rooms though so that might be the issue.
Also, i would image this kind of workflow may present some issues where a room has more than 1 ceiling.
I will have a look on using it with a larger project but it would be pushing my limited Dynamo skills
If I’m not mistaken, with your script all rooms get one and the same value?
Here is my Revit project where you’ll find 3 rooms with 3 different ceiling height and as result I want a room parameter where I find the value of the height of the ceiling: test dynamo hoogte ruimtes.rvt (1.2 MB)
OK, I encountered a problem with getting some of the ceilings in the Room.BoundingElements node (from Archi-Lab) from the data’s collector nodes but I managed it with Select All Rooms. Here is where I am: the list of rooms should now match the list of ceilings and their corresponding heights: