Skip to content
This repository was archived by the owner on Sep 26, 2025. It is now read-only.

Try to calculate HR in zones based on moving time, not elapsed #40#42

Merged
stancld merged 2 commits intomasterfrom
bugfix/hr-processing-moving-time
Jun 21, 2025
Merged

Try to calculate HR in zones based on moving time, not elapsed #40#42
stancld merged 2 commits intomasterfrom
bugfix/hr-processing-moving-time

Conversation

@stancld
Copy link
Owner

@stancld stancld commented Jun 21, 2025

Closes #40.

@stancld stancld force-pushed the bugfix/hr-processing-moving-time branch from 259a111 to f4b3195 Compare June 21, 2025 14:22
@codecov-commenter
Copy link

codecov-commenter commented Jun 21, 2025

Codecov Report

Attention: Patch coverage is 88.88889% with 3 lines in your changes missing coverage. Please review.

Project coverage is 62.82%. Comparing base (dc5d34b) to head (2e47cd4).

Files with missing lines Patch % Lines
backend/api/hr_processing.py 95.65% 1 Missing ⚠️
backend/api/models.py 0.00% 1 Missing ⚠️
backend/api/worker.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
+ Coverage   62.63%   62.82%   +0.19%     
==========================================
  Files          16       16              
  Lines        1365     1372       +7     
==========================================
+ Hits          855      862       +7     
  Misses        510      510              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@stancld stancld force-pushed the bugfix/hr-processing-moving-time branch from f4b3195 to 2e47cd4 Compare June 21, 2025 14:26
@stancld stancld changed the title Try to calculate HR in zones based on moving time, not elapsed Try to calculate HR in zones based on moving time, not elapsed #40 Jun 21, 2025
@stancld stancld merged commit 3948e9b into master Jun 21, 2025
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bugfix: Use moving time instead of elapsed time

2 participants