tt-acm / EnergyAnalysisForDynamo

Dynamo <> Autodesk Green Building Studio interoperability
Other
29 stars 18 forks source link

Separate Zones based on orientation #66

Closed mostaphaRoudsari closed 9 years ago

mostaphaRoudsari commented 9 years ago

Similar to https://github.com/tt-acm/EnergyAnalysisForDynamo/issues/65 but this time for orientations.

bhowes-tt commented 9 years ago

I dig it, @mostaphaRoudsari. One question - How is orientation determined for a zone with multiple exterior faces?

mostaphaRoudsari commented 9 years ago

@bhowes-tt if you do the zoning right, you normally shouldn't end up with zones like that but in can still happen in some cases. My approach in Honeybee is to make a separate list for them and let the user decides what they want to do with them > Zones with faces only to orientation a, Zones with faces only to orientation b, zones with faces to both orientation a and b.

bhowes-tt commented 9 years ago

Nice. I've certainly seen some zones coming out of Vasari with multiple faces. If we were doing everything inside of a component, would we just choose the largest face?

Or were you thinking of doing this as a cluster or a sample file? We might already be able to do this with the decompoze zone and surface.vector nodes...

mostaphaRoudsari commented 9 years ago

I think they happen in Revit and not Vasari. I might be wrong though.

Selecting the biggest face is tricky. It is hard to know what will happen with that specially in case of auto-zoning that the user doesn't have that much control on the zones. I still like the idea of having them in a separate group and let the user decide about them.

I see them as couple of nodes. It should be easy to do so since as you said we have the methods already. I see them as couple of nodes that let you manage the zones > separate by orientation, separate by level, separate by program, etc. I think they will be all pretty useful. I'm not sure when is the right time to add them but I know that they will be useful.

mostaphaRoudsari commented 9 years ago

Now that we have decompose zone maybe this should be separate surfaces based on orientation? Then having zones with multiple walls facing different directions will be already addressed! I will check rest of issues including this one on Monday.

mostaphaRoudsari commented 9 years ago

With combination of this node and separate by levels users can do interesting facade studies. Here is an example to change glazing constructions. @arkdanielnielsen I guess you will like this for your studies.

image

PS1: I found 2 bugs in Vasari zoning playing around with these new set of nodes:

  1. Sometimes it generates window between interior walls.
  2. Some of the floors are wrongly assigned as roof.

PS2: DesignScript's vector.AngleBetween doesn't work as expected so I had to write a private method to handle that for now. I've already reported this.