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

Enhancement: Intersect and Surface Matching #75

Closed
Ski90Moo opened this issue May 26, 2021 · 4 comments · Fixed by #113
Closed

Enhancement: Intersect and Surface Matching #75

Ski90Moo opened this issue May 26, 2021 · 4 comments · Fixed by #113

Comments

@Ski90Moo
Copy link
Contributor

The intersect and surface matching seems to work way better when you do it space-by-space instead of trying to do the whole model at the same time. I am wondering if it is possible to write a user script that does this function for the whole model, but it only executes intersect and match one space at a time?

@macumber
Copy link
Collaborator

@macumber
Copy link
Collaborator

You could try sorting the spaces to intersect in a better order since the intersection results are dependent on the order they happen, we did that here to some extent:

https://github.com/NREL/OpenStudio/blob/develop/src/model/Space.cpp#L3289
https://github.com/NREL/OpenStudio/blob/develop/src/model/Space.cpp#L2223

@Ski90Moo
Copy link
Contributor Author

Ok; for some reason the script seems to always freeze when surface matching the entire model, or if there are a very large number of spaces. There always seems to be errors associated with matching some surfaces. It seems if there are too many errors, then SketchUp just freezes. I am wondering if there is a way for the script to detect an error and stop running before it tries to surface match additional spaces?

@macumber
Copy link
Collaborator

It probably could, the problem is identifying the situation that indicates an error has (or better is likely to) occur

This was referenced Mar 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants