Skip to content

Explain movement-masked stress hours - #2248

Open
kavemang wants to merge 2 commits into
ryanbr:mainfrom
kavemang:fix/stress-masked-hours-copy-2125
Open

kavemang wants to merge 2 commits into
ryanbr:mainfrom
kavemang:fix/stress-masked-hours-copy-2125

Conversation

@kavemang

Copy link
Copy Markdown

Summary

  • show the localized activity-masked hour count on the Stress timeline and hosted Today stress card on Android and Apple
  • replace Android's partly hardcoded timeline caption with complete day-relative and personal-baseline resources
  • describe the daytime signal without claiming that the currently disabled daytime HRV input is active
  • carry Android's canonical non-overlapping masked-hour count into the Today card so half-hour display windows cannot inflate it

Verification

  • cd android && ./gradlew :app:compileFullDebugKotlin
  • cd android && ./gradlew :app:testFullDebugUnitTest --tests 'com.noop.analytics.DaytimeStressTest' --tests 'com.noop.widget.StressTraceTest' --rerun-tasks
  • python3 Tools/i18n_audit.py --ci upstream/main
  • python3 Tools/doc_comment_lint.py
  • JSON and Android XML resource validation

Apple app targets were not compiled locally because this development host is Linux; the app-target changes are covered by upstream Apple CI.

Closes #2125

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stress copy: the "you were moving" count is computed on both platforms and rendered nowhere

1 participant