Skip to content

Commit

Permalink
fix: use median in impute_days for cadence only
Browse files Browse the repository at this point in the history
  • Loading branch information
chanshing committed May 4, 2024
1 parent 957e801 commit 87e090c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/stepcount/stepcount.py
Original file line number Diff line number Diff line change
Expand Up @@ -501,9 +501,9 @@ def _cadence_p95(x, steptol, walktol=30):
# adjusted estimates first form a 7-day representative week before final aggregation
# TODO: 7-day padding for shorter recordings
# TODO: maybe impute output daily_cadence? but skip user-excluded days
day_of_week_cadence_peak1 = impute_days(daily_cadence_peak1).groupby(daily_cadence_peak1.index.weekday).median()
day_of_week_cadence_peak30 = impute_days(daily_cadence_peak30).groupby(daily_cadence_peak30.index.weekday).median()
day_of_week_cadence_p95 = impute_days(daily_cadence_p95).groupby(daily_cadence_p95.index.weekday).median()
day_of_week_cadence_peak1 = impute_days(daily_cadence_peak1, method='median').groupby(daily_cadence_peak1.index.weekday).median()
day_of_week_cadence_peak30 = impute_days(daily_cadence_peak30, method='median').groupby(daily_cadence_peak30.index.weekday).median()
day_of_week_cadence_p95 = impute_days(daily_cadence_p95, method='median').groupby(daily_cadence_p95.index.weekday).median()

cadence_peak1 = day_of_week_cadence_peak1.median()
cadence_peak30 = day_of_week_cadence_peak30.median()
Expand Down

0 comments on commit 87e090c

Please sign in to comment.