backup control to dos #146
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@kendallbaertlein: Can you change this PR to be merging your changes into dev, rather than main? I can help you with this if you're not sure how to do it. |
|
Next steps: @jmaguire1 to review first, then ask Michael to review. |
jmaguire1
reviewed
Dec 9, 2024
| @@ -17,14 +17,16 @@ | |||
|
|
|||
| # Timing parameters | |||
| 'start_time': dt.datetime(2018, 1, 1, 0, 0), # year, month, day, hour, minute | |||
| 'time_res': dt.timedelta(minutes=10), # time resolution of the simulation | |||
There was a problem hiding this comment.
@kendallbaertlein: We'll want to revert these changes, they were just for testing on your local branch. I don't think we actually want anything in run_dwelling.py to change here.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
added in to do items for the backup control
[✔️] Enable offset between setpoint and when backup ER comes on.
[✔️] Add max temperature for backup operation ("lockout temperature"), default to 35 F
[✔️] Enable lockout of backup after a setpoint change: assume hard lockout for 10 minutes, after 10 minutes if temp is trending in the right direction stay locked out for full 30 minutes, otherwise override lockout.
[ ] bonus: staged backup
Parametric study:
[ ] Baseline is what we think a default ecobee thermostat would do
[ ] Case that reflects a customer who values comfort over savings (enable backup more, smaller offset before backup turns on
[ ] Case that reflects a customer who values savings over comfort (lower temp where compressor is enabled, bigger offset)
For parametric, look at energy consumption, utility bills, and unmet loads. We'll need to finalize the building properties and sizing assumptions, but we want a reasonable case with a single speed ASHP installed in Denver metro as a retrofit on existing buildings.
Bonus "baseline" for the parametric is where we started in OCHRE vs. Ecobee. That's at least useful for us, more debatable if it belongs in a publication.