renamed "find_change_points_with_scipy" to "find_change_points" for maintainability
This commit is contained in:
parent
afea54b0e6
commit
2b3b567a31
2 changed files with 2 additions and 2 deletions
|
|
@ -98,7 +98,7 @@ class CO2FormData(FormData):
|
||||||
if not re.compile("^(2[0-3]|[01]?[0-9]):([0-5]?[0-9])$").match(time):
|
if not re.compile("^(2[0-3]|[01]?[0-9]):([0-5]?[0-9])$").match(time):
|
||||||
raise TypeError(f'Wrong time format - "HH:MM". Got "{time}".')
|
raise TypeError(f'Wrong time format - "HH:MM". Got "{time}".')
|
||||||
|
|
||||||
def find_change_points_with_scipy(self) -> list:
|
def find_change_points(self) -> list:
|
||||||
"""
|
"""
|
||||||
Perform change point detection using scipy library (find_peaks method) with rolling average of data.
|
Perform change point detection using scipy library (find_peaks method) with rolling average of data.
|
||||||
Incorporate existing state change candidates and adjust the result accordingly.
|
Incorporate existing state change candidates and adjust the result accordingly.
|
||||||
|
|
|
||||||
|
|
@ -25,7 +25,7 @@ class CO2ReportGenerator:
|
||||||
elif isinstance(CO2model.number, IntPiecewiseConstant):
|
elif isinstance(CO2model.number, IntPiecewiseConstant):
|
||||||
occupancy_transition_times = list(CO2model.number.transition_times)
|
occupancy_transition_times = list(CO2model.number.transition_times)
|
||||||
|
|
||||||
ventilation_transition_times: list = form.find_change_points_with_scipy()
|
ventilation_transition_times: list = form.find_change_points()
|
||||||
# The entire ventilation changes consider the initial and final occupancy state change
|
# The entire ventilation changes consider the initial and final occupancy state change
|
||||||
all_vent_transition_times: list = sorted(
|
all_vent_transition_times: list = sorted(
|
||||||
[occupancy_transition_times[0]] +
|
[occupancy_transition_times[0]] +
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue