Skip to content

Python: define WorkoutState with the FIT intensity values. - #28

Merged
sanderroosendaal merged 7 commits into
mainfrom
python/laps-and-intervals
Oct 14, 2026
Merged

sanderroosendaal merged 7 commits into
mainfrom
python/laps-and-intervals

Conversation

@sanderroosendaal

@sanderroosendaal sanderroosendaal commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

Unknown values are read as Other. When a record disagrees with its lap intensity, the lap wins.

Co-authored-by: Cursor cursoragent@cursor.com

Caller laps without intensity are rejected. A session with no laps is written as one active lap.

Co-authored-by: Cursor cursoragent@cursor.com

Implements the Python side of #23. Also implements the Python side of #39 (moving time is every lap except rest). Should be merged after #23 and #39.

The diff against main contains only this change.

🤖 Generated with Claude Code

Sander Roosendaal and others added 4 commits October 7, 2026 09:31
Unknown values are read as Other. When a record disagrees with its lap intensity, the lap wins.

Co-authored-by: Cursor <cursoragent@cursor.com>
Caller laps without intensity are rejected. A session with no laps is written as one active lap.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Implements the Python side of the moving-time rule in #39. Lap gains
total_timer_s, which is read and written, and RowingSession.moving_time_s()
sums it over every lap except rest. validate warns when a record in a rest
lap carries rowing data.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…vals

# Conflicts:
#	python/tests/test_roundtrip.py
@sanderroosendaal
sanderroosendaal merged commit 072eeaf into main Oct 14, 2026
4 checks passed
@sanderroosendaal
sanderroosendaal deleted the python/laps-and-intervals branch October 14, 2026 15:33
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.

2 participants