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

Solar Simulations should use trimmed surface #714

Open
christophwaibel opened this issue Jan 11, 2022 · 3 comments
Open

Solar Simulations should use trimmed surface #714

christophwaibel opened this issue Jan 11, 2022 · 3 comments
Assignees
Labels
bug 👾 a bug!!!

Comments

@christophwaibel
Copy link
Contributor

Describe the bug
When conducting solar simulations, the original untrimmed surface geometry is used. This is very unintuitive for the user, because the user usually only sees the trimmed geometry. The problem becomes visible when displaying the coloured mesh with irradiation values. See screenshot below

Expected behavior
Simulation should by default work with the trimmed geometry,

Screenshots
image

Software

  • Rhino Version 7
  • Hive Version 1.14
@ChrisZenhub
Copy link

related issue #626

@philipschulz
Copy link
Collaborator

philipschulz commented Jul 5, 2023

@ChrisZenhub this seems to be caused simply by the component that generates the mesh from the referenced surface, as that apparently ignores the trim and makes a mesh from the underlying untrimmed surface. Frustratingly, setting the bool option "Allow faces to overhang trims" to False on that component doesn't seem to fix this behaviour in the expected way...

Image

@christophwaibel
Copy link
Contributor Author

hmm ok thanks @philipschulz . can you propose an alternative workflow for the template, where this doesn't happen? Or maybe make a new gh component that ensures that always the trimmed geometry is used? Or, actually what I prefer: implement it in the solar model that it always uses the trimmed geometry (the one, that is shown on the rhino display)

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

No branches or pull requests

4 participants