Skip to content

Update VAVTurndownDuringReheat Untested outcome #59

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

Open
yunjoonjung-PNNL wants to merge 11 commits into
base: develop
Choose a base branch
from
Open

Update VAVTurndownDuringReheat Untested outcome #59

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
35e9104
Merge branch 'VAV_turndown_using_average' of https://github.com/pnnl/…
yunjoonjung-PNNL Nov 7, 2024
72d2bef
Merge branch 'develop' of https://github.com/pnnl/ConStrain into VAV_…
yunjoonjung-PNNL Nov 7, 2024
2cc22f6
Merge branch 'develop' of https://github.com/pnnl/ConStrain into VAV_…
yunjoonjung-PNNL Nov 8, 2024
71053f5
Updated the vav turndown during reheat untest condition
yunjoonjung-PNNL Nov 8, 2024
a8638a7
Merge branch 'develop' of https://github.com/pnnl/ConStrain into VAV_…
yunjoonjung-PNNL Nov 8, 2024
4a5f96d
Added a new unit test
yunjoonjung-PNNL Nov 8, 2024
a17dc48
Updated library file
yunjoonjung-PNNL Nov 8, 2024
919d182
Updated VAVTurndownDuringReheat logic & plotting
yunjoonjung-PNNL Dec 5, 2024
6a82de5
Added unit test for untested case
yunjoonjung-PNNL Dec 5, 2024
81e2634
Reverted changes in checklib.py
yunjoonjung-PNNL Dec 6, 2024
b7e93ab
Merge branch 'develop' into VAV_turndown_testing
leijerry888 Mar 30, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions constrain/library/vav_turndown_during_reheat.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,8 @@

"""

import logging

from constrain.checklib import RuleCheckBase


Expand All @@ -44,13 +46,11 @@ class VAVTurndownDuringReheat(RuleCheckBase):
]

def verify(self):
# Make sure every value in `V_dot_VAV_max` is greater than 0
assert (
self.df["V_dot_VAV_max"] > 0
).all(), "Not all `V_dot_VAV_max` values are greater than 0"

# Check if the `reheat_coil_flag` column has only False values
if (self.df["reheat_coil_flag"] == False).all():
# Check if the `reheat_coil_flag` column has only True/False value
if self.df["reheat_coil_flag"].nunique() == 1:
self.df["result"] = "Untested"
elif (self.df["V_dot_VAV_max"] == 0).any():
logging.error("Any `V_dot_VAV_max` value shouldn't be zero.")
self.df["result"] = "Untested"
else:
self.df["V_dot_VAV_ratio"] = self.df["V_dot_VAV"] / self.df["V_dot_VAV_max"]
Expand Down
2 changes: 1 addition & 1 deletion constrain/schema/library.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1602,7 +1602,7 @@
"description_verification_type": "procedure-based",
"assertions_type": "pass",
"description_assertions": [
"if (reheat_coil_flag == False).all():",
"if reheat_coil_flag.nunique() == 1:",
" Untested",
"else: ",
" V_dot_VAV_ratio = V_dot_VAV/V_dot_VAV_max",
Expand Down
76 changes: 75 additions & 1 deletion tests/test_vav_turndown_during_reheat.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ def test_vav_turndown_during_reheat_fail(self):
binary_result = verification_obj.check_bool()
self.assertFalse(binary_result)

def test_vav_turndown_during_reheat_untested(self):
def test_vav_turndown_during_reheat_no_reheat_untested(self):
points = [
"reheat_coil_flag",
"V_dot_VAV",
Expand Down Expand Up @@ -127,3 +127,77 @@ def test_vav_turndown_during_reheat_untested(self):

self.assertEqual(results, expected_results)
self.assertEqual(verification_obj.check_bool(), "Untested")

def test_vav_turndown_during_reheat_all_reheat_untested(self):
points = [
"reheat_coil_flag",
"V_dot_VAV",
"V_dot_VAV_max",
]

timestamp = [
datetime(2024, 8, 1, 12, 0, 0),
datetime(2024, 8, 1, 13, 0, 0),
datetime(2024, 8, 1, 14, 0, 0),
datetime(2024, 8, 1, 15, 0, 0),
]

data = [
[True, 350, 620],
[True, 370, 620],
[True, 360, 620],
[True, 380, 620],
]

df = pd.DataFrame(data, columns=points, index=timestamp)

verification_obj = run_test_verification_with_data(
"VAVTurndownDuringReheat", df
)
results = list(verification_obj.result)
expected_results = [
"Untested",
"Untested",
"Untested",
"Untested",
]

self.assertEqual(results, expected_results)
self.assertEqual(verification_obj.check_bool(), "Untested")

def test_vav_turndown_during_reheat_zero_V_dot_VAV_max_untested(self):
points = [
"reheat_coil_flag",
"V_dot_VAV",
"V_dot_VAV_max",
]

timestamp = [
datetime(2024, 8, 1, 12, 0, 0),
datetime(2024, 8, 1, 13, 0, 0),
datetime(2024, 8, 1, 14, 0, 0),
datetime(2024, 8, 1, 15, 0, 0),
]

data = [
[True, 350, 0],
[True, 370, 0],
Copy link
Collaborator

@leijerry888 leijerry888 Dec 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems like this is the only difference from the previous test. Should the error log being asserted in the test?

Copy link
Collaborator Author

@yunjoonjung-PNNL yunjoonjung-PNNL Dec 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point.

One thing I don't understand is that when I added the following code at the end of the unit test (line 284), the logobs is empty, which I don't understand. I'll keep this PR for now to take more time to find the reason. This shouldn't impact the testing.

with self.assertLogs() as logobs:
    self.assertEqual(
        "ERROR:root:Any `V_dot_VAV_max` value shouldn't be zero.",
        logobs.output[0],
    )

[True, 360, 620],
[True, 380, 620],
]

df = pd.DataFrame(data, columns=points, index=timestamp)

verification_obj = run_test_verification_with_data(
"VAVTurndownDuringReheat", df
)
results = list(verification_obj.result)
expected_results = [
"Untested",
"Untested",
"Untested",
"Untested",
]

self.assertEqual(results, expected_results)
self.assertEqual(verification_obj.check_bool(), "Untested")
Loading