SplitBuildingMass not able to split a simple box into zones for more than 1 floor - Part 2

Maki · February 17, 2021, 10:09am

Dear All
It is my first topic. and I wish somebody can answer my question.
I am reffering to @ElzineBraasch previous post. I tried to use the latest component SplitBuildingMass2Floors. however it does not work?

There are two error messages. The message says

One of the masses is causing a problem. Check the output for the mass that causes the problem. You should consider breaking up this mass into smaller pieces.
Solution exception:too many values to unpack

I can see there are similar topics about the component. Please let me know how I could divide the simple shoe box model into small zones.

Thanks.

https://drive.google.com/file/d/1Nt02nsTZWtn0RrVGYM2rcbMDBS5-v01S/view?usp=sharing

AbrahamYezioro · February 17, 2021, 4:37pm

I noticed that indeed the component fails in Rhino7. Just checked in R6 and it works fine.
So this is another component that is not compatible.
I think it is important to fix it since is an essential part of the modeling for those using HB Legacy, including myself.
@chris, any insights?
@wim, maybe this needs to be addressed on Rhino’s side?

Thanks,
-A.

maheshjayayachander · February 22, 2021, 11:31am

Hey @AbrahamYezioro, looks like this issue with Rhino7 & Split2Floor component has been solved with a custom component. Look at this link [Modified component SplitBuildingMass2Floors, to split complex buildings].
also @chris, what are your thoughts?

AbrahamYezioro · February 22, 2021, 9:42pm

Thanks @maheshjayayachander,
It works indeed. Though i don’t like that it works on an old and customized component instead of the native HB one.
The good news is, i suppose, that it is fixable. The bad news … if it will be fixed.
Something for @chris to consider.
-A.

SaeranVasanthakumar · February 22, 2021, 10:27pm

I would take a look at it… but I don’t have R7 yet. I should just bite the bullet and upgrade.

AbrahamYezioro · February 23, 2021, 3:13am

Thanks @SaeranVasanthakumar!
You can try the 90 days free … before any bullet, -A.

SaeranVasanthakumar · February 23, 2021, 6:49am

@AbrahamYezioro good idea! I just pushed this PR with a fix.

And if @miraclema is still on the forums, your modified component looks useful. If you are interested in contributing I suggest reading up on LB’s contributing guidelines, and submitting code to github. It’s hard to evaluate code changes on the forums (just keep in mind legacy components are being phased out…).

AbrahamYezioro · March 2, 2021, 4:45am

Thanks @saeran!!
Still waiting for the push to be approved, but didn’t want to wait to thank you.
Best,
-A.

AbrahamYezioro · March 3, 2021, 6:07am

Now i can thank you @SaeranVasanthakumar!
It is working just fine.
-A.

SaeranVasanthakumar · March 3, 2021, 6:44am

Awesome, glad to hear it @AbrahamYezioro!

maheshjayayachander · March 3, 2021, 12:41pm

That was quite a quick update, thanks a ton @SaeranVasanthakumar @AbrahamYezioro

chris · March 4, 2021, 2:46am

I just wanted to say thank you, @SaeranVasanthakumar , for the fix and I’m sorry that it took me a few days to see the PR and merge it. I’ll try to be more on top of it going forward and you are always welcome to message me for cases like this where we have a PR pending.