Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wishlist: Open water swimming data series #91

Open
alevin opened this issue Oct 2, 2022 · 3 comments
Open

Wishlist: Open water swimming data series #91

alevin opened this issue Oct 2, 2022 · 3 comments
Assignees

Comments

@alevin
Copy link

alevin commented Oct 2, 2022

Please consider adding an option to use time per 100 (yards/meters) as a pace unit.
imho it is more suitable for swimming.

I noticed that this unit is used for "Swim pace" data series, but it is not visible for "Open Water swimming"
I would be great to add it and other swimming metrics ( SWOLF , strokes/min , strokes/length) , to "Open Water Swimming"

Thanks

@alex-hhh alex-hhh self-assigned this Oct 3, 2022
@alex-hhh
Copy link
Owner

alex-hhh commented Oct 3, 2022

"strokes/min" is really "cadence" and this graph should be available if your device records this data. "SWOLF" and "strokes/length" do not make sense for Open Water Swimming (OWS) since they are related to the length of the swimming pool (and the values don't compare across pools of different length). For OWS there is also a "Stride" graph, which is "distance/stroke".

However, I did want to add a pace measured in min/100 m (or yards), but the internal data representation makes this a bit more complicated, but I will look into it a bit more.

alex-hhh added a commit that referenced this issue Oct 8, 2022
... OWS activities are treated specially (same as Lap Swimming ones) in the
session inspector.

NOTE: this introduces some problems in trend charts for pace series, since the
"pace" series in data frames is in seconds/100m (or 100 yards)
@alex-hhh
Copy link
Owner

alex-hhh commented Oct 8, 2022

I pushed some changes to show Open Water Swimming pace as min/100m or min/100yd, let me know if this works for you.

@alevin
Copy link
Author

alevin commented Oct 11, 2022

Thanks, pace looks good to me !
About "SWOLF", and "strokes/length" - I agree that they make less sense for the open water, but still can be calculated for the intervals and Garmin is showing them in mobile version of Connect, but not in the browser (not sure why).
imho, this metrics could be still valuable addition in the laps projection, especially when recalculated for different split types ( 100m, best effort ... )

Thanks, again !

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

No branches or pull requests

2 participants