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

intersectSurfaces having inconsistent results on measure test for model articulation #2560

Closed
DavidGoldwasser opened this issue Apr 7, 2017 · 5 comments

Comments

@DavidGoldwasser
Copy link
Collaborator

DavidGoldwasser commented Apr 7, 2017

I run the same test three times, and get three different results. This test is update to model articulation to allow partial top floors by entering 5.5 vs 6 for number of above grade stories.

In this run of test the right side split non-convex surfaces, and didn't intersect or split non-convex on the left.

screen shot 2017-04-07 at 1 23 12 pm

In another test got similar results although while the right side intersected it didn't split non-convex

screen shot 2017-04-07 at 1 23 54 pm

Another time also similar to the first. In this case everything on the right did what it should the left side did intersect but didn't split non-convex

screen shot 2017-04-07 at 1 24 25 pm

@macumber I haven't committed this code, but I can point you to test and provide OSM's to you.

@macumber
Copy link
Contributor

macumber commented Apr 7, 2017

Bummer, I have some hope that upgrading boost might fix some of these things.

My first question will be why are you generating this weird top floor that doesn't match the other floors footprints?

@macumber
Copy link
Contributor

macumber commented Apr 7, 2017

Another option is to do some sorting of spaces and surfaces before matching, that will at least make this deterministic.

@DavidGoldwasser
Copy link
Collaborator Author

DavidGoldwasser commented Apr 7, 2017

I updated the number of stories above grade to take any double (1 or greater) vs. just positive integers. This was done for project to model specific tenants from a building that may span full and partial stories

This test happens to use 5.5 stories above grade. My current logic just reduces the floor area of the top story while maintaining the same aspect ratio and x,y origin. I could just change the width or height but not both.

I'm not currently sorting the space going into this vector so I can try that.
OpenStudio::Model.intersectSurfaces(spaces)

I thought about running this a few hundred times (via a script) and store metrics of results by looking at surface count and matched surface count, although at list on a dozer or so runs after sorting the spaces in the vector, it provides consistent results (matching the third picture). I'm currently just running test in ruby, and not through CLI. I don't think that would make a difference.

Issue #2527 with joinAll popped in my head as something potentially related. It was order specific.

@gokmartian
Copy link

I think I have a similar version of this problem. My model has two buildings. Different floors of the buildings don't have similar floor plans, as above. Intersection and surface matching completely removes surfaces from one of the surfaces, and creates holes. The other matching surface, as a result, is set to outdoors/roofcieling/ground, instead of getting matched to the adjacent surface. Different holes in other surfaces are created when I intersect and match again.

@jmarrec
Copy link
Collaborator

jmarrec commented Jun 8, 2021

@DavidGoldwasser Could you try and reproduce with 3.2.1 or provide models? #4221 may have fixed this... Closing for now, can reopen if not fixed and with enough information (the models) to reproduce.

@jmarrec jmarrec closed this as completed Jun 8, 2021
jmarrec added a commit that referenced this issue Apr 3, 2023
```
1/4 Test #2562: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago .......................   Passed    1.39 sec
2/4 Test #2563: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago_GeometryDiagnostics ...   Passed    1.40 sec
3/4 Test #2560: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete ...................................   Passed    3.79 sec
4/4 Test #2561: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete_GeometryDiagnostics ...............   Passed    3.95 sec
```
jmarrec added a commit that referenced this issue Apr 6, 2023
```
1/4 Test #2562: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago .......................   Passed    1.39 sec
2/4 Test #2563: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago_GeometryDiagnostics ...   Passed    1.40 sec
3/4 Test #2560: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete ...................................   Passed    3.79 sec
4/4 Test #2561: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete_GeometryDiagnostics ...............   Passed    3.95 sec
```
jmarrec added a commit that referenced this issue Apr 11, 2023
```
1/4 Test #2562: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago .......................   Passed    1.39 sec
2/4 Test #2563: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago_GeometryDiagnostics ...   Passed    1.40 sec
3/4 Test #2560: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete ...................................   Passed    3.79 sec
4/4 Test #2561: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete_GeometryDiagnostics ...............   Passed    3.95 sec
```
jmarrec added a commit that referenced this issue Apr 24, 2023
```
1/4 Test #2562: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago .......................   Passed    1.39 sec
2/4 Test #2563: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago_GeometryDiagnostics ...   Passed    1.40 sec
3/4 Test #2560: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete ...................................   Passed    3.79 sec
4/4 Test #2561: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete_GeometryDiagnostics ...............   Passed    3.95 sec
```
jmarrec added a commit that referenced this issue May 10, 2023
```
1/4 Test #2562: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago .......................   Passed    1.39 sec
2/4 Test #2563: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago_GeometryDiagnostics ...   Passed    1.40 sec
3/4 Test #2560: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete ...................................   Passed    3.79 sec
4/4 Test #2561: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete_GeometryDiagnostics ...............   Passed    3.95 sec
```
jmarrec added a commit that referenced this issue May 16, 2023
```
1/4 Test #2562: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago .......................   Passed    1.39 sec
2/4 Test #2563: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago_GeometryDiagnostics ...   Passed    1.40 sec
3/4 Test #2560: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete ...................................   Passed    3.79 sec
4/4 Test #2561: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete_GeometryDiagnostics ...............   Passed    3.95 sec
```
jmarrec added a commit that referenced this issue Jun 12, 2023
```
1/4 Test #2562: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago .......................   Passed    1.39 sec
2/4 Test #2563: ModelFixture.ThreeJSForwardTranslator_RefBldgOutPatientNew2004_Chicago_GeometryDiagnostics ...   Passed    1.40 sec
3/4 Test #2560: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete ...................................   Passed    3.79 sec
4/4 Test #2561: ModelFixture.ThreeJSForwardTranslator_7_7_Windows_Complete_GeometryDiagnostics ...............   Passed    3.95 sec
```
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

5 participants