Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improve the runtime performance of ViewFactor #3719

Merged
merged 1 commit into from
Oct 10, 2019

Conversation

kbenne
Copy link
Contributor

@kbenne kbenne commented Oct 10, 2019

close #3713

OpenStudio Pull Request Template

Please read OpenStudio Pull Requests to better understand the OpenStrudio Pull Request protocol.

Time for the test case reference in #3713 is now

kbenne-27067s:OpenStudio kbenne$ time ./build/Products/openstudio build/test.rb
49 view factors => 0.0 sec
64 view factors => 0.0 sec
81 view factors => 0.01 sec
100 view factors => 0.01 sec
121 view factors => 0.01 sec
144 view factors => 0.02 sec
169 view factors => 0.02 sec
196 view factors => 0.04 sec
225 view factors => 0.05 sec
256 view factors => 0.06 sec
289 view factors => 0.08 sec
324 view factors => 0.11 sec
361 view factors => 0.14 sec
400 view factors => 0.18 sec
441 view factors => 0.23 sec
484 view factors => 0.3 sec
529 view factors => 0.4 sec
576 view factors => 0.49 sec
625 view factors => 0.64 sec
676 view factors => 0.76 sec

We can get better but it will require some lower level workspace effort that I will work on and put in a different commit after release.

@kbenne kbenne merged commit 4be3662 into develop Oct 10, 2019
jmarrec added a commit to jmarrec/OpenStudio that referenced this pull request Oct 10, 2019
@jmarrec
Copy link
Collaborator

jmarrec commented Oct 10, 2019

Nice job @kbenne, but this lacks comments. We're doing unusual stuff here, so comments would be nice, added some in #3720

@jmarrec jmarrec deleted the 3713-ViewFactorPerformance branch October 10, 2019 06:30
kbenne added a commit that referenced this pull request Oct 16, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OS 2.9.0 addViewFactor is very slow
2 participants