pcolby / bipolar

Free your data from Polar FlowSync
GNU General Public License v3.0
115 stars 16 forks source link

FlowSync 3.0 compatibility #89

Closed pcolby closed 5 years ago

pcolby commented 5 years ago

As reported by @zdhons in https://github.com/pcolby/bipolar/issues/80#issuecomment-431324125, FlowSync 3.0.0 is no longer compatible with the hook version packaged with Bipolar. We need to update the hook to match whatever FlowSync is using now.

I've already checked, and seen that FlowSync on MacOS X uses Qt 5.5.1. Will check Windows next.

Reminder: we track FlowSync's Qt versions at https://github.com/pcolby/bipolar/wiki/Hooks#qt-versions.

pcolby commented 5 years ago

@zdhons, can I confirm that you are using a Mac? It appears that FlowSync 3.0 has moved the OSX version to using Qt 5.5.1, but FlowSync was (and still is) using Qt 5.5.1 since version 2.6.2, so that doesn't appear to have changed (on Windows).

?

Thanks :)

ArtMag commented 5 years ago

I must update Polar FlowSync 3.0. When I changed oryginal hook Qt5Network.dll (2015) on your (12-2017), Polar FlowSync didn`t work. I used Windows 7.0.

pcolby commented 5 years ago

Thanks everyone. I'll take a look at a Windows host later today.

pcolby commented 5 years ago

Ok, so the most obvious changes for FlowSync 2.6.2 (last version for Windows) to 3.0.0 on Windows:

  1. moved VC runtimes from msvc100 to msvc120 (ie Visual C++ 2010 to 2013)
  2. updated OpenSSL from 1.0.1g to 1.0.2o

The first one is most likely the cause of the crash (to be expected). The second change should still be binary compatibly, but since I'll have to update the hook to match the VC runtime, I'll update the OpenSSL version to match too.

pcolby commented 5 years ago

Ok, there's an updated Q5Network.dll available. There's no code changes, just rebuilt with VS 2013 and OpenSSL 1.0.2p - it fixes the crash :)

You can install it manually for now - just drop the dll into the FlowSync directory. I'll release an updated full Bipolar installer later (probably not till tomorrow).

If you install it manually, let me know how it goes for you :)

Thanks!

ArtMag commented 5 years ago

Thank you for new hook and fast reaction. Hook works well :) It is important for me because I am doing research in Nepal now. I have hundreds hours data from many V800 and only 24 h for synchronisation with PolarFlow. New upgrade PolarFlowSync suprise me. I know that I can download all data from Polar Flow (RR data also) in JONS-format but Bipolar have better for me formats. Thank you again.

zdhons commented 5 years ago

Paul, thanks very much for your prompt help! I could not use my FirstBeat Athlete without RR-data. Do you think about the Bipolar service for new Polar Vantage?

Best regards

Zdenek


From: Paul Colby notifications@github.com Sent: Sunday, October 21, 2018 5:44:13 AM To: pcolby/bipolar Cc: zdhons; Mention Subject: Re: [pcolby/bipolar] FlowSync 3.0 compatibility (#89)

Ok, there's an updated Q5Network.dll available. There's no code changes, just rebuilt with VS 2013 and OpenSSL 1.0.2p - it fixes the crash :)

You can install it manually for now - just drop the dll into the FlowSync directory. I'll release an updated full Bipolar installer later (probably not till tomorrow).

If you install it manually, let me know how it goes for you :)

Thanks!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/pcolby/bipolar/issues/89#issuecomment-431636371, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AermCyLMmFhLInJAxwYBcTZbaw2428E4ks5um-2NgaJpZM4XwJby.

roesassi commented 5 years ago

@pcolby thanks a lot. I also tried the new dll and it worked as a charm. If it might be of any help, I noticed that in the AppData\Local\Polar\PolarFlowSync\export folder, the hook saved a file which I never saw in the last three years, named v2-users-275367-activities. It is probably in preparation of the new Vantage series.

pcolby commented 5 years ago

Bipolar 0.5.9 has now been released, including the update hook :)