philosowaffle / peloton-to-garmin

Convert workout data from Peloton into JSON/TCX/FIT files and automatically upload to Garmin Connect
https://philosowaffle.github.io/peloton-to-garmin/
GNU General Public License v3.0
284 stars 375 forks source link

[Question] Training Status no longer updated? #605

Closed gunjanmarwah closed 3 months ago

gunjanmarwah commented 10 months ago

Anyone else finding today that Peloton activities synced into Garmin with HR and a solid effort are not appearing in the Training Status & Training Load? I did a 30 minute ride that I expect to be contributing to me Low Aerobic load, and a few others (even if they don't contribute; they should still appear). None are even appearing for today in the Exercise Load. Is it just me? Is it Garmin? Or are they cracking down on these imports?

mgsimon commented 10 months ago

I think this is resolved with #578 already. Did you try >= v4.1.0?

gunjanmarwah commented 10 months ago

Oh no I completely missed that. Scanned through the issues first and didn't see it. I don't think I did it right though as it's not working - are you able to help take a peek @mgsimon ? .github/workflows/sync_peloton_to_garmin.yml Also, any idea how far back this went? Thinking I will have to delete and resync activities.

mgsimon commented 10 months ago

12/21, stable image should work I think.

gunjanmarwah commented 10 months ago

Didn't work... /__w/_temp/a60cc903-6e61-42cd-abe4-b3dff52e5455.sh: 1: /app/PelotonToGarminConsole: not found Error: Process completed with exit code 127.

I am not at all a dev, please help! :(

gunjanmarwah commented 10 months ago

I changed it from v3.6.1 to v4.1.0 then console-latest and then console-stable. None worked. All the same error.

mgsimon commented 10 months ago

I'm using docker and not sure but the run command in the master branch is ...

- run: /app/ConsoleClient

Maybe there are some changes in the meantime.

gunjanmarwah commented 10 months ago

Got it, thanks. @philosowaffle any suggestions?

philosowaffle commented 10 months ago

When you setup the Secrets on your Repository did you happen to setup a custom DEVICE_INFO secret?

gunjanmarwah commented 10 months ago

I did not; that is empty.

On Tue, 9 Jan 2024 at 18:45, Bailey @.***> wrote:

When you setup the Secrets https://philosowaffle.github.io/peloton-to-garmin/latest/install/github-action/#secrets on your Repository did you happen to setup a custom DEVICE_INFO secret?

— Reply to this email directly, view it on GitHub https://github.com/philosowaffle/peloton-to-garmin/issues/605#issuecomment-1883969004, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOI6564GAGCBE63H4KZ5SQLYNXJARAVCNFSM6AAAAABBSEVGWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBTHE3DSMBQGQ . You are receiving this because you authored the thread.Message ID: @.***>

gunjanmarwah commented 10 months ago

@philosowaffle I've set up the Device Info now. Reran on stable, then latest, then 4.1.0. None worked, same error "/__w/_temp/9a8f2659-d352-4e8b-a69a-cdfe6ef04023.sh: 1: /app/PelotonToGarminConsole: not found ". Put it back to v3.6.1 and it's syncing again but still not contributing to the training load. The only thing that has really changed now is that I have Device Info probably haha.

philosowaffle commented 10 months ago

Alright, let's try the following.

  1. Switch this line to use: philosowaffle/peloton-to-garmin:console-v4
  2. Switch this line to use: /app/ConsoleClient instead
gunjanmarwah commented 10 months ago

@philosowaffle tried to do that. It brought in activities but they are still not showing in the Training Load.

philosowaffle commented 10 months ago

Ok, Let's stay on this configuration as this will atleast ensure you get future updates/fixes when they happen.

Several people noticed that after this issue was fixed, the next activity they synced earned them the First Tacx Ride badge on Garmin. Did you happen see something similar? (trying to confirm your activities are getting uploaded with the correct device)

gunjanmarwah commented 10 months ago

Okay. No I don't see anything like that. I was hoping for these to be synced over properly because I'm in a training plan again. Is it on the Garmin side?

On Wed, Jan 10, 2024, 7:38 p.m. Bailey @.***> wrote:

Ok, Let's stay on this configuration as this will atleast ensure you get future updates/fixes when they happen.

Several people noticed that after this issue was fixed, the next activity they synced earned them the First Tacx Ride badge on Garmin. Did you happen see something similar? (trying to confirm your activities are getting uploaded with the correct device)

— Reply to this email directly, view it on GitHub https://github.com/philosowaffle/peloton-to-garmin/issues/605#issuecomment-1885989687, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOI6563HTS25ELVGUBV2K23YN4X7VAVCNFSM6AAAAABBSEVGWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBVHE4DSNRYG4 . You are receiving this because you authored the thread.Message ID: @.***>

philosowaffle commented 10 months ago

To be honest, I'm not really sure what's going on, I just have a few more things we can try to see if anything makes a difference. Unfortunately, this is a bit of trial and error since I'm not able to reproduce on my end and nothing stands out in the logs.

first thing to try:

  1. I think you still have a custom DeviceInfo being provided in your latest runs, let's delete that and try another sync. I know you tried this previously, I just want to establish a baseline before trying the next step.
  2. Remove DeviceInfo secret, the run a sync and see if anything changes in terms of the Device shown on Garmin and TE/TS on the workout

next:

  1. I noticed you have lines 48/49 set to True. Let's try setting both of these to false. I worry that Garmin may not be giving you credit because P2G is providing this data instead of letting Garmin calculate it.
  2. Run sync again and see if anything changes. Also let me know if you still get the "Time in Zones" charts in the workout in Garmin.
gunjanmarwah commented 10 months ago

Thanks I will try these shortly! For the Device Info - am I just clearing out the secret or do I have to make any update to the code also? Previously the secret existed but was blank. Should I revert it to that?

On Sat, Jan 13, 2024, 9:17 a.m. Bailey @.***> wrote:

To be honest, I'm not really sure what's going on, I just have a few more things we can try to see if anything makes a difference. Unfortunately, this is a bit of trial and error since I'm not able to reproduce on my end and nothing stands out in the logs. first thing to try:

  1. I think you still have a custom DeviceInfo being provided in your latest runs, let's delete that and try another sync. I know you tried this previously, I just want to establish a baseline before trying the next step.
  2. Remove DeviceInfo secret, the run a sync and see if anything changes in terms of the Device shown on Garmin and TE/TS on the workout

next:

  1. I noticed you have lines 48/49 https://github.com/gunjanmarwah/peloton-to-garmin/blob/master/.github/workflows/sync_peloton_to_garmin.yml#L48 set to True. Let's try setting both of these to false. I worry that Garmin may not be giving you credit because P2G is providing this data instead of letting Garmin calculate it.
  2. Run sync again and see if anything changes. Also let me know if you still get the "Time in Zones" charts in the workout in Garmin.

— Reply to this email directly, view it on GitHub https://github.com/philosowaffle/peloton-to-garmin/issues/605#issuecomment-1890470837, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOI6567ROAXZZMYFTVWQLNTYOKJOFAVCNFSM6AAAAABBSEVGWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQGQ3TAOBTG4 . You are receiving this because you authored the thread.Message ID: @.***>

philosowaffle commented 10 months ago

Just deleting the secret from the GitHub secrets list should be sufficient. 👍

gunjanmarwah commented 10 months ago

Ok there's some movement! I did both things you suggested and I'm afraid I wasn't necessarily testing the right cases as something definitely changed but I was testing against any of ride, strengths or yoga activities. Thing is my last ride was only Jan 8 and that was the one that made me go down this rabbit hole. I think this update from Garmin is looking only for activities either tracked directly on Garmin (I had tested a strength activity tracked directly on Garmin vs a Just Cardio on Peloton) or those with a distance. I went all the way back to activities since and including the ride after changing lines 48/49, deleted them, and resynced them. The ride is showing up now in the exercise load, including the charts. None of the other activities are showing in the load. Previously they would still show, even if they didn't have a contribution. I'm not convinced that none of them should have had a contribution, so I'll keep an eye on this. I'll update you here over the coming weeks? Thanks again for all your help!

On Sat, Jan 13, 2024, 8:29 p.m. Bailey @.***> wrote:

Just deleting the secret from the GitHub secrets list should be sufficient. 👍

— Reply to this email directly, view it on GitHub https://github.com/philosowaffle/peloton-to-garmin/issues/605#issuecomment-1890815834, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOI6564LET5LNBU55ZLYNA3YOMYGJAVCNFSM6AAAAABBSEVGWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQHAYTKOBTGQ . You are receiving this because you authored the thread.Message ID: @.***>