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

Intersection issue found when using create bar #4372

Closed
DavidGoldwasser opened this issue Jul 2, 2021 · 4 comments · Fixed by #4527
Closed

Intersection issue found when using create bar #4372

DavidGoldwasser opened this issue Jul 2, 2021 · 4 comments · Fixed by #4527

Comments

@DavidGoldwasser
Copy link
Collaborator

Issue overview

There is small identified error in pre/post intersection area, but that doesn't account for larger error in expected exterior perimeter reported.

        "stdout" : "[openstudio.model.Surface] <1> Initial area of other surface 'Surface 316' 57.2551 does not equal post intersection area 57.1732\n[openstudio.model.Surface] <1> Initial area of surface 'Surface 326' 13.9355 does not equal post intersection area 13.8945\n[openstudio.model.Surface] <1> Initial area of surface 'Surface 338' 13.9355 does not equal post intersection area 13.8945\n",
        "step_errors" : 
        [
           "East/West walls don't have the expected area (actual 5,492.4418 ft^2, target 3,661.6279 ft^2)"
        ],

Current Behavior

That error is called here
https://github.com/NREL/openstudio-extension-gem/blob/develop/lib/openstudio/extension/core/os_lib_model_generation.rb#L2543

Expected Behavior

Expected values should match and meaasure and method shoudl run without an error

Steps to Reproduce

I'm going to add a test on branch of extension gem with these inputs and should be able to replicate it

Possible Solution

Details

Environment

Some additional details about your environment for this issue (if relevant):

  • Platform (Operating system, version):
  • Version of OpenStudio (if using an intermediate build, include SHA):

Context

Is blocking URBANopt project until I figure out work arond

@DavidGoldwasser DavidGoldwasser added the Triage Issue needs to be assessed and labeled, further information on reported might be needed label Jul 2, 2021
@tijcolem tijcolem added severity - Normal Bug and removed Triage Issue needs to be assessed and labeled, further information on reported might be needed labels Jul 9, 2021
@tijcolem tijcolem added this to the OpenStudio SDK 3.3.0 milestone Jul 9, 2021
@tijcolem tijcolem assigned lgentile and unassigned jmarrec Jul 9, 2021
@tijcolem tijcolem assigned ggartside and unassigned lgentile Aug 2, 2021
@ggartside
Copy link
Collaborator

@DavidGoldwasser Did you ever add a test on a branch of extension gems to reproduce this issue? I'm looking for something I can run in open studio to reproduce this issue before I dig any deeper

@DavidGoldwasser
Copy link
Collaborator Author

DavidGoldwasser commented Sep 26, 2021

@ggartside I think this is the branch of extension gem I was working on.
https://github.com/NREL/openstudio-extension-gem/compare/intersect_tests_072021

I'll test it out tomorrow or Monday with the same installer as #4431

@DavidGoldwasser
Copy link
Collaborator Author

@ggartside sorry for my delay, this issue in extension gem explains things in more detail. I just ran test with version of OpenStudio used from the intersection PR and I'm still seeing this behavior.
NREL/openstudio-extension-gem#128

I run the test by doing bundle exec rake spec in the extension gem after first doing bundle install. This is on the intersext_tests_072021 branch and not develop. I have a meeting in a few minutes but I can email you 1 or 2 OSM files that are the output of the test, which you can use to test it without having to checkout the extension gem.

DavidGoldwasser added a commit to NREL/openstudio-extension-gem that referenced this issue Oct 10, 2021
This fully bypasses both intersection and matching which bypasses NREL/OpenStudio#4372 for now but it still needs to be fixed. I' need to clean this up a bit later tonight, and then address last failing test.
@DavidGoldwasser
Copy link
Collaborator Author

DavidGoldwasser commented Oct 11, 2021

@ggartside and @tijcolem this is still an issue with the 3.3 alpha installer, however for use with our model articulation I enhanced the logic when make_mid_story_surfaces_adiabatic to fully bypass intersection and matching. So fix for core OS doesn't need to go into 3.3. Later tonight I'll do PR on this branch of Extension Gem, but I'm trying to work on one more failing test. The code hi-lighted below differentiates interior vs. exterior walls base don simple use case oof rectangular building shape.
https://github.com/NREL/openstudio-extension-gem/blob/intersect_tests_072021/lib/openstudio/extension/core/os_lib_model_generation.rb#L1199-L1256

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

Successfully merging a pull request may close this issue.

5 participants