Skip to content

Support for Soleus Watches and Strava OAuth API#33

Open
kbb29 wants to merge 8 commits intomlt:masterfrom
kbb29:master
Open

Support for Soleus Watches and Strava OAuth API#33
kbb29 wants to merge 8 commits intomlt:masterfrom
kbb29:master

Conversation

@kbb29
Copy link

@kbb29 kbb29 commented Apr 30, 2016

Hi,

Not sure if anything has been done with this project recently, but I have found it useful for uploading stuff from my Soleus watch. I have added support for downloading data in the format used by my watch, and I have added a tcx writer (based on the original csv2tcx.py). Finally, I have added a --strava option to download.py, to upload the tcx files to Strava using their OAuth API. Hope this is useful. If not, feel free to ignore...

Keith

@mlt
Copy link
Owner

mlt commented May 2, 2016

Hey! Thank you for working on this! Strava was on TODO list, though historically I just got stuck with MapMyRun as it just works. The project went into hiatus shortly after I started reorganizing the code base to make it more library alike. (Although my view is that it should silently upload everything to whatever has been set up without pressing any buttons.) So I still do have some backlog that I need to clean up and to port your changes over.

I'd have to review changes first as it looks like it is quite invasive. Also, please, create a ticket next time before starting to work on that many changes :-) Perhaps it would have been a good motivation for me to finish up migration earlier.

Do you have any binary dump you could upload somewhere (--debug)? I'd like to add soleus test to new code base.

""" Append point to a database """
self.ptsWriter.writerow(point)

def commit(self):
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I kept commit for SQL writers and to mimic behavior otherwise.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, yeah can't remember why I got rid of this. Probably just because I thought we didn't need it anymore with the way I had rewritten the code.

@kbb29
Copy link
Author

kbb29 commented May 2, 2016

And thank you for working on this! If I hadn't found your code, I would have had to install Windows somewhere just to get data off my watch... I must admit, when I started working on this, my goal was to make it do what I wanted (ie get data from a soleus watch and upload it to strava) so submitting a pull request was a bit of an afterthought. So I'm happy to help with merging the code with changes you have made since (presumably, whatever is on the 'dev' branch?).

I have uploaded a dump of two tracks from my watch here:
http://www.filedropper.com/schwinn810

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