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.

Sign up for GitHub

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

Jump to bottom

Release 0.2.0 #108

Merged
merged 958 commits into from
Jan 28, 2021
Merged

Release 0.2.0 #108

merged 958 commits into from
Jan 28, 2021

Conversation

corymosiman12
Copy link
Contributor

No description provided.

corymosiman12 and others added 30 commits October 27, 2020 12:20
@corymosiman12
Merge pull request #90 from BuildingSync/fix/update-gemspec
a87c1e4 
update: gemspec include openstudio-ee
@corymosiman12
update: gemfile
34e9d54
@TMaile
adding more documentation
c617098
@TMaile
Merge branch 'DA_PullReq56_Old' into DA_Update
1118139
@TMaile
adding more documentation
fb92c56
@TMaile
adding more documentation
2d65703
@TMaile
adjusting the dependencies again
30c4567
@TMaile
adjusting the dependencies again
c6fbb8d
@TMaile
adjusting the dependencies again
aab1dac
@TMaile
adjusting the dependencies again
37ddab2
@TMaile
Merge branch 'DA_Update' into DA_Update_RefSched
21929f8 
# Conflicts:
#	lib/buildingsync/makers/workflow_maker.rb
@TMaile
using the bldgsync_measures branch in openstudio-ee again
1726a37
@TMaile
Merge remote-tracking branch 'origin/DA_PullReq56_Old-triage' into DA…
a95d8d8 
…_PullReq56_Old
@TMaile
using the bldgsync_measures branch in openstudio-ee again
c43894d
@TMaile
Merge branch 'DA_PullReq56_Old' into DA_Update
b0ad96f 
# Conflicts:
#	Gemfile
@TMaile
using the bldgsync_measures branch in openstudio-ee again
3305da2
@TMaile
supporting location specific parameters on the site and building obje…
cd33b9b 
…ct now, including climate zone, resolves #89
@corymosiman12
update: lock dependencies to official releases
20051e5
@corymosiman12
Merge branch 'DA_Update' into update/other-gems
0eb2f63
@TMaile
update readme and added more in-code documentation
ed3f1aa
@TMaile
fixing the missing end tag in Gemfile
26046e1
@TMaile
fixing issues with a missing parameter
b021bb5
@TMaile
Merge pull request #92 from BuildingSync/update/other-gems
ccf3fd2 
update/other-gems
@TMaile
removed read_city_and_state_name function since it was called twice
22c131d
@TMaile
Merge pull request #91 from BuildingSync/DA_Update_RefSched_CZ
da1a3d9 
Enable reading of climate zone (and other parameters) at the building level in addition to the site level
@TMaile
adding documentation to the helper classes
31e4228
@TMaile
Update README.md
41cc18b 
cleanuup
@TMaile
adjust_schedules_new taken from typical_building_from_model seems to …
7f99c40 
…work, needs finetuning
@TMaile
update on the schedule adjustments
a1ce5d4
@TMaile
update on the schedule adjustments
fd00033
@corymosiman12 corymosiman12 requested a review from nllong January 22, 2021 23:05
@corymosiman12 corymosiman12 self-assigned this Jan 22, 2021
@corymosiman12
Copy link
Contributor Author

There are currently 4 tests that are not passing. They are as follows:

  • Occupancy Type Spec: 1 failure / 14. The failure occurs on the SmallHotel, which is captured by Fix: OccupancyTypeSpec SmallHotel #103
  • Loads System Spec: 1 failure / 3. The failure occurs with the schedule adjustment, which is captured by Fix LoadSystem Spec Expectations #109
  • Translator Scenario Simulation spec: 2 failures / 8. This has to do with the fact that certain measures are not working as expected on Building 151, which are captured in the translator_spec and also by Broken Measures #107.

The recommendation is to create a release for this version even though not all tests are passing. They will be addressed in subsequent patch releases.

@corymosiman12 corymosiman12 marked this pull request as ready for review January 28, 2021 00:37
@nllong nllong merged commit 107c9e7 into master Jan 28, 2021
@nllong nllong deleted the prep/0.2.0 branch January 28, 2021 15:45
@corymosiman12 corymosiman12 restored the prep/0.2.0 branch January 28, 2021 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nllong nllong Awaiting requested review from nllong

Assignees

@corymosiman12 corymosiman12

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@corymosiman12 @nllong @TMaile @vtnate @TShapinsky