New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
'Honeybee_Glazing based on ratio' crashes Grasshopper/Rhino #572
Comments
@saeranv , |
@chriswmackey that's surprising, do you have any idea what it could be in the splitBldgMass component that would be causing the issue (i.e. is it to do with the floor height splitting or to do with the perimeter core split? I'm a little wary about trying to debug this component since debugging code that crashes Rhino/GH is a nightmare, but I can try and keep an eye out while dealing with the splitBldgMass components for issues. |
@chriswmackey @saeranv @mostaphaRoudsari I did some investigation with the profiler, what ever is happening it is increasing the processing time in a strange fashion. With 10 honeybee zones: With 20 honeybee zones With 30 honeybee zones with 50 honeybee zones It crashes at around 90 zones, I am going to dig into the code next week with the python profiler tool to try and see where the hold up is occurring, Chris or Mostapha have you seen something like this before? |
The plot thickens I re opened the file and now the process time in both the 20 hb zone case and the 50 hb zone case has halved! I think this a deeper issue to do with deep copys of zones, @mostaphaRoudsari what are your thoughts on this? I'd like to get to the bottom of this if it can be fixed/ improved upon |
Hi everyone, I'm not sure this is completely relevant but I am also facing some problems with the combination of split building mass and glazing based on ratio components. I keep getting that annoying "outward facing angle of subsurface differs more than 90 degrees" error that crashes my E+ simulation. Passing the same zones without adding glazing seems to work. I hope there's some work around for this. Edit: Not sure this is at all relevant but it seems to be the ground zones failing the E+. Regards, |
@antonszilasi profiling is a different issue and you should open a new issue for that one. My suggestion is to do the same study on Rhino WIP and Grasshopper I. I suspect that GHPython component itself has a lot to do with this issue. Check this post. |
@antonszilasi , |
Ah so each time you run a Gh component the code is compiled? so if you change the code it has to compile again? |
@antonszilasi , |
@saeranv and @antonszilasi , @mostaphaRoudsari and I finally figured out what was causing this (and other issues with large Honeybee models) a few weeks ago. It was the result of a runaway memory process that was happening after you tried to edit/copy zones that had solved adjacencies. Specifically, after solved adjacencies, each subsequent component would copy all adjacent boundary surfaces when it copied the zones. This on its own would be fine but, because each adjacent surface also has a parent zone, a parent zone would be copied for each adjacent surface. And those parent zones would have other child surfaces and the whole thing just blew up your memory when you have more than a few zones Needles to say, we have fixed this now and all problems associated with this issue should go away: Thanks for pointing us towards this and you've helped solve a problem for a lot of people. |
We were using |
When I add glazing to my HBZone using the Glazing based on ratio component - Rhino is immediately crashing. It will work if I'm generating a 1-5 floors (with 5 zones each) but if I try to generate glazing for an entire tower (as I am in the sample gh file) - I'm getting an immediate crash.
Here's a link to my forum post with a gh file reproducing the problem.
http://www.grasshopper3d.com/group/ladybug/forum/topics/honeybee-glazing-based-on-ratio-crashes-rhino-for-large-amount-1
The text was updated successfully, but these errors were encountered: