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

added option to create .gpx in batch #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mikesinouye
Copy link

Swapped phones and my Samsung account sync failed, thankfully I had downloaded the json data beforehand just in case.

I had hundreds of workouts I wanted to transfer away from S Health, and the tool seemed to only support a single workout at a time. Added a checkbox that allows merging every 2 json into a separate .gpx file. With the checkbox disabled, it should function the same as before. Not sure if I saved time but maybe someone else will benefit?

@hpmmatuska
Copy link

Hello.

I just switch from Samsung to Garmin. I would like to convert all of it on batch.

Whatever I try, it's do batch of 10 activities at once (I need to convert couple of hundreds). Where is the gap?
Would be possible to do a check if each file pair is present, otherwise ignore particular input entry?

@mikesinouye
Copy link
Author

Hey, unfortunately I really threw this together, so it detects pairs by file upload order. It relies on the files being uploaded in alphabetical order.

I'm not sure why it breaks after 10 - I definitely had a lot more than 10. I took a quick look, and nothing jumped out at me as to why that might be happening.

If certain workouts are missing their pairs, and this is sporadic in your file set, you'll have to rework this code a bit. Maybe use a map, keyed on filename, to detect any pairs? Then after iterating over the full list you can pass the map to the actual function that creates the gpx. This would also resolve the not-so-intuitive alphabetical upload requirement.

Good luck! Hope you can get it working

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