search

mgifos / quick-plan

Defines and schedules Garmin workouts
Apache License 2.0
137 stars 23 forks source link

Scheduling of previously defined workouts #20

Closed leingang closed 6 years ago

leingang commented 6 years ago

It seems like if I have a cell in my CSV file such as:

running: 8E
- run: 8mi
- cooldown: lap-button

and then a later cell with just 8E in it, the expected effect is that the "8E" workout should be defined, then scheduled on both days. The Ultra 80k model CSV file has several repeated workout days like this (e.g., "8k jog" each Wednesday).

Instead, I get it scheduled on the day for which it's defined, but nothing gets scheduled on the later day. Am I doing it wrong?

mgifos commented 6 years ago

Can you attach a complete CSV file to the issue, so I can test it myself?

mystygage commented 6 years ago

I had a similar issue with the training on first Wednesday here (6k @GA1/2+STL): https://docs.google.com/spreadsheets/d/1EXa9QL2AvyIkZIFI239Hf2QPO2ugehQ97-HXvdO4vgg/edit?usp=sharing

It was defined for the day before, but quick-plan didn’t schedule it for Wednesday. Two weeks later it was also scheduled and here everything worked fine.

mgifos commented 6 years ago

Sounds like a problem with asynchronous execution ... e.g. a definition of 6k @GA1/2+STL has not been completed before the scheduling of the first repetition has started. I'm gonna investigate this further by the code staring.

mgifos commented 6 years ago

I managed to reproduce the issue. The bug is confirmed. The problem is if you try to schedule a workout defined in a same week again.

leingang commented 6 years ago

Thank you! I look forward to a new release so I can take advantage of this fix.

mgifos commented 6 years ago

Released in v0.2.4