RebarContainer.ByBars

[EvanMurphy17]

Hello,

I am attempting to create a container containing a layout created with Rebar.SetLayoutAsMaximumSpacing, and the output is an Element. However, the RebarContainer.ByBars node only accepts Rebar[]. I was wondering if it's possible to allow the RebarContainer.ByBars node to accept Elements.

Thank you in advance.

Evan

[moethu]

https://github.com/tt-acm/DynamoForRebar/blob/master/src/DynamoRebar/Revit/RebarContainer.cs#L388-L391
should be

            List<Revit.Elements.Rebar> mybars = new List<Revit.Elements.Rebar>();
            foreach (Revit.Elements.Element e in rebars)
            {
                if (e.InternalElement is Autodesk.Revit.DB.Structure.Rebar)
                {
                    Revit.Elements.Rebar bar = Revit.Elements.Rebar.FromExisting(e.InternalElement as Autodesk.Revit.DB.Structure.Rebar, true);
                    mybars.Add(bar);
                }
            }
            return new RebarContainer(mybars);

[moethu]

actually the output is: Revit.Elements.UnknownElement

[moethu]

@EvanMurphy17 please try: https://we.tl/t-PpfFw8nFi8

[EvanMurphy17]

@moethu worked like a charm!

Thank you so much. Really appreciate all you do.

[moethu]

thanks, I'll let @eertugrul know to release a new version.

[moethu]

@eertugrul could you release an update of this dynamo package please? see changes above

[EvanMurphy17]

One (possible) issue:

I am not sure if I am misunderstanding what the Container.ByBars node does, but when I just have the rebar placed it looks like this:

When I take the rebar elements and feed them into the Container.ByBars node, it outputs this:

The rebar I had already created using ByCurve --> SetLayoutAsMaximumSpacing is there, but then there are duplicated instances in the wrong spots, that are rebar containers. Not sure if I am just misunderstanding the node, or if something is happening that is not intended.

Thanks for your help

[moethu]

@EvanMurphy17 sorry, I didn't see you question until now. I think you shouldn't create container bybars - that's only if you already got all your bars, but you want to create them by layputAsMaximumSpacing, right?