mgifos / quick-plan

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

Nothing gets imported... :( #22

Closed f4hla closed 6 years ago

f4hla commented 6 years ago

Hi Here is my plan, I try to schedule and get all to null (best case) (or various random errors) -- HELP :) --- Command ./quick-plan schedule -n 2018-10-18 -x -e xxx@xxx.xxx -p XXX -v /Users/gilou/Dropbox/Maison/Sports/GARMIN_REUNION.csv2 -d ---Log Successfully logged in to Garmin Connect

Deleting workouts:

Creating workouts:

Scheduling:

Statistics: 0 deleted 0 imported 0 scheduled Logged out and closed connection Bye

---Plan (attached file as well) GARMIN_REUNION.txt

Week;Monday;Tuesday;Wednesday;Thursday;Friday;Saturday;Sunday 1;;"running: VMA-Asc-15%

1h30Fond-Vallone-PPG";"running: Long-2hvtt+1h30-500D+

";1hFond-Vallone-PPG;"running: Sensation-Plat -warmup: 20:00@z2

running: VMA-trail

running: End-Musc-Squat

1h30Fond-Vallone-PPG";"running: RandoTrail-5h-2kD+

mgifos commented 6 years ago

Your CSV file is not valid. You can validate it here: https://csvlint.io/

and this is the status of the validation I did on your file: https://csvlint.io/validation/5b5b56cb2869ac0004000001

Did you edit your CSV manually? Which tool did you use to export?

f4hla commented 6 years ago

Hi, thanks for the reply, but I think there is something else (the validation report shows invalid csv due to the use of ';' iso ','). Anyway, you can see the new simplifier csv here, pass with 0 error your link and still same result, nothing imported... Any help? Regards GARMIN_REUNION_Soir.txt

-- Successfully logged in to Garmin Connect

Deleting workouts:

Creating workouts:

Scheduling:

Statistics: 0 deleted 0 imported 0 scheduled Logged out and closed connection Bye

f4hla commented 6 years ago

by the way, also getting this error randomly [WARN] [07/28/2018 09:51:22.244] [quick-plan-akka.actor.default-dispatcher-12] [akka.actor.ActorSystemImpl(quick-plan)] Illegal header: Illegal 'server' header: Invalid input 'EOI', expected product or comment (line 1, column 1): ^

mgifos commented 6 years ago

Hi, The last warning is ok, it's not an error. I'm not at home this weekend so I can only comment from my phone, but I'll check your file thoroughly on Monday. By then, please check if you have any special character in the workout names and be sure you are using the latest version of the software because the previous one only recognised "workout" keyword instead of "running" or "cycling". Supported characters of workout names are only those from ASCII table up to 128.

f4hla commented 6 years ago

Hi thanks for your support (and enjoy the weekend ;))

I am using 0.2.3 release, so should be ok. In the simplified file I just uploaded, no special char.

Thanks a lot

PS : I also attach the formatted result from csvlint.io so you can test on something not questionable GARMIN_REUNION_Soir.txt

mgifos commented 6 years ago

A short instruction list till then. Try with a smaller portions of your plan, e.g. try specifying just a first week of your plan with a few simple workouts and simple names. Make sure the format of your spreadsheet is the same as the one in the example. I haven't use any other spreadsheet editor but the one from Google, so to be on the safe side, try first with that one and we can focus later on supporting the others.

mgifos commented 6 years ago

One more thing, I used this CSV parser: https://github.com/tototoshi/scala-csv

It could be that the issue is even related to this library because from the output of the first version I noticed that it couldn't parse the CSV.

f4hla commented 6 years ago

Hi ok - I was using csv export from M$ Excel... I switched to Google Editor online then download as csv and...it works :) Great!! I guess then there maybe a few things added by xls. You may want to check but at least every one can now know that through Google online editor it works. thanks (but I would appreciate a little test from inside using the previously posted file). Regards

mgifos commented 6 years ago

In your first example of CSV, MS Excel enforces \r carriage return and Google editor enforces line feed \n as the line break character. I'll open another issue to support this.