garmin-connect-export submodule not pushed to github about garmindb

[tcgoetz]

Which one? Whats the error?

[earlyday]

The last commit two days ago on GarminDB depends on a non existing sha in garmin-connect-export. mån 23 apr. 2018 kl. 22:57 skrev Tom Goetz <[email protected]>:

…

Which one? Whats the error? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#11 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/Ak4T7-_PN7pJ0gXf0LaVAQg6tJW0A-lSks5trkA8gaJpZM4TgmCF> .

[tcgoetz]

Submodules are a PITA. Everything looks right from my side:

orion-wl:garmin-connect-export tgoetz$ git log
commit 85da1a10296e94d38d785e9f756d3cb10dc3407e (HEAD -> pull_req_42)
Author: Tom Goetz <[email protected]>
Date:   Sat Apr 21 19:26:03 2018 -0400

    port orig repo pull req 42 to add support for new API

commit ab949368b065aa05530566ffa255290e01b0605f (origin/pr/4)
Author: Kyle Krafka <[email protected]>
Date:   Tue Dec 22 22:26:24 2015 -0500

    A few fixes after testing a complete export
...

and

orion-wl:garmin-connect-export tgoetz$ git push
Everything up-to-date
orion-wl:garmin-connect-export tgoetz$ git status
On branch pull_req_42

I'll look at it some more.

[earlyday]

This is very strange to me. I don't understand this. I cloned again and now I get the right stuff. My first attempt resulted in this history, which is different from what I see after a new clone. I will carry on with make setup but get compiler errors. commit 93b24c29bac316fe2b8e5888f4d8f81a2769e8a9 Author: Tom Goetz <[email protected]> Date: Tue Mar 27 09:04:24 2018 -0400 handle json data with no summary gcexport.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) commit df66b0b8eb2ea7e6a542582e9e82f012c543378c Author: Tom Goetz <[email protected]> Date: Sun Mar 4 14:31:07 2018 -0500 log saving json data gcexport.py | 1 + 1 file changed, 1 insertion(+) commit 87f0d101000e90ad590ddb526544138f704833b9 Author: Tom Goetz <[email protected]> Date: Sat Feb 24 09:09:32 2018 -0500 remove debugging exit gcexport.py | 1 - old/garmin-connect-export.php | 245 ---------------------------------------------------------------------------------------------------------------------------------- 2 files changed, 246 deletions(-) 2018-04-24 2:50 GMT+02:00 Tom Goetz <[email protected]>:

…

Submodules are a PITA. Everything looks right from my side: orion-wl:garmin-connect-export tgoetz$ git log commit 85da1a10296e94d38d785e9f756d3cb10dc3407e (HEAD -> pull_req_42) Author: Tom Goetz ***@***.***> Date: Sat Apr 21 19:26:03 2018 -0400 port orig repo pull req 42 to add support for new API commit ab949368b065aa05530566ffa255290e01b0605f (origin/pr/4) Author: Kyle Krafka ***@***.***> Date: Tue Dec 22 22:26:24 2015 -0500 A few fixes after testing a complete export ... and orion-wl:garmin-connect-export tgoetz$ git push Everything up-to-date orion-wl:garmin-connect-export tgoetz$ git status On branch pull_req_42 I'll look at it some more. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#11 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/Ak4T7xgQUuaSeIHAG74KnNEkAjvTw80Oks5trna9gaJpZM4TgmCF> .

[tcgoetz]

I will carry on with make setup but get compiler errors.

What compiler errors? Not sure what should be compiling. Think it's all Python.

[earlyday]

I very much appreciate the effort you do. I have tried in various ways and the compilation happens in tcxparser. I managed to go around this and can now run gcexport.py but the scrape_garmin.py requires a working firefoxdriver and selenium. Apparently, my install don't have the webdriver.xpi and after fixing it I end up in more errors about missing x_ignore_nofocus.so. I guess this require some effort to make it work on debian stretch. What os are you running? /Erland 2018-04-24 13:42 GMT+02:00 Tom Goetz <[email protected]>:

…

I will carry on with make setup but get compiler errors. What compiler errors? Not sure what should be compiling. Think it's all Python. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#11 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/Ak4T70U7Bxfw5PX5pEx9wa3xLyH8w2EPks5trw-dgaJpZM4TgmCF> .

[tcgoetz]

I'm on OSX.

There is code in the "make deps" to install the Firefox driver for Linux, but it hasn't been tested. If you can get it working, please submit a patch.

[earlyday]

Hi, I solved some by installing selenium using pip, instead of apt-get. Later I was required to add headless option as firefox option. I could not push to you repo so pick up the patch here:
https://github.com/earlyday/sandbox/blob/master/headless.patch

[tcgoetz]

I've updated the code to download weight and monitoring files via the GC API. No more scraping. No more Firefox needed.

[earlyday]

Testing, but missing commit 0c0b0a57ee6b084b2a03c448add8b01382eaca15 in Fit. 2018-04-30 3:36 GMT+02:00 Tom Goetz <[email protected]>:

…

I've updated the code to download weight and monitoring files via the GC API. No more scraping. No more Firefox needed. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#11 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/Ak4T79JnKCUjHWAiR1OQfI7JTl6yPIgWks5ttmq4gaJpZM4TgmCF> .

[tcgoetz]

Try it again.

[earlyday]

Perfect. tis 1 maj 2018 kl. 13:53 skrev Tom Goetz <[email protected]>:

…

Try it again. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#11 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/Ak4T75ADkQJfOMDfZmhiNvf8M-WKZnSGks5tuEzHgaJpZM4TgmCF> .

[tcgoetz]

Is everything working for you now? Can we close this bug?

[earlyday]

Yes consider it solved. Thank you. tors 3 maj 2018 kl. 16:36 skrev Tom Goetz <[email protected]>:

…

Is everything working for you now? Can we close this bug? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#11 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/Ak4T7_FraV4KgaZn8a1LekTC4lW4ODrHks5tuxXZgaJpZM4TgmCF> .