-
Notifications
You must be signed in to change notification settings - Fork 5
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
Additional tests to check flexibility options of thermal house and storage #825
Conversation
…l storage and house above target temp
…se below target temp
…ouse # Conflicts: # CHANGELOG.md
…ouse # Conflicts: # CHANGELOG.md
…ouse # Conflicts: # CHANGELOG.md
…ouse # Conflicts: # CHANGELOG.md
…ouse # Conflicts: # CHANGELOG.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Over all it looks good to me! Just some minor points.
isRunning = false, | ||
0, | ||
Some(hpData.ambientTemperature), | ||
Kilowatts(0.0), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is the expression changed from the double value?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you refer to the case lefthand where the lastHouseState was 21°C and 80 Kilowatts feedin, this one should still be in the table (labeled Storage and house have remaining capacity
)
src/test/scala/edu/ie3/simona/model/participant/HpModelSpec.scala
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some questions and comments from my side
src/test/scala/edu/ie3/simona/model/participant/HpModelSpec.scala
Outdated
Show resolved
Hide resolved
src/test/scala/edu/ie3/simona/model/participant/HpModelSpec.scala
Outdated
Show resolved
Hide resolved
src/test/scala/edu/ie3/simona/model/participant/HpModelSpec.scala
Outdated
Show resolved
Hide resolved
src/test/scala/edu/ie3/simona/model/participant/HpModelSpec.scala
Outdated
Show resolved
Hide resolved
This reverts commit f279b89.
include suggestion from reviewer Co-authored-by: Sebastian Peter <[email protected]>
…-house' into df/#729-additional-tests-thermal-house
Signed-off-by: Sebastian Peter <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, all good if my proposed change is fine for you too.
resolves #729