Skip to content
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

Intermediate floor\ceiling boundary conditions #205

Closed
xavfa opened this issue Nov 9, 2020 · 4 comments
Closed

Intermediate floor\ceiling boundary conditions #205

xavfa opened this issue Nov 9, 2020 · 4 comments

Comments

@xavfa
Copy link

xavfa commented Nov 9, 2020

Hi,
creating several zones per floor seem to work fine, but I found that boundary conditions for ceilings and floor were fixed to outdoor with sun and wind exposed for the ceiling and wind exposes for the floor. The ceiling of the lower floor should be connected to the floor of the upper one. I guess this rely on the matched \ unmatched surface routine but I am not sure at all.
Please is this an error from my side or boundary condition for intermediate storey with the add_block function is missing something ?
Thanks in advance

@xavfa
Copy link
Author

xavfa commented Nov 9, 2020

Hi,
it is something to do with the almostequal function in utilities.py
I have manage to solve this issue making a rotating routine of the polygone s.coords to check if almost equal to m.coords.
The boundary for Tutorial 3 works now as well for me.
Thanks in advance to confirm the mistakes (or to point out my mistake in using the package)

@amylee-lixinyi
Copy link

Hi,
I seems to come across the same issue of unmatched ceiling/floor as you did, could you please enlighten me how did you manage to fix this ?
Thanks.

@xavfa
Copy link
Author

xavfa commented Apr 20, 2021

Hi, have a look at the issue posted just before.
non convex polygons #204
I should have post also here the solution. Sorry.
As said, an even better on is on the development branch.
Best.

@xavfa
Copy link
Author

xavfa commented Apr 20, 2021

This issue could could be closed now though

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants