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

Output gpx and tcx files are not splitted on separate tracks #1

Open
knedlyk opened this issue Nov 10, 2014 · 10 comments
Open

Output gpx and tcx files are not splitted on separate tracks #1

knedlyk opened this issue Nov 10, 2014 · 10 comments

Comments

@knedlyk
Copy link

knedlyk commented Nov 10, 2014

It is not an issue, it is just a feature request. When Crane watch has more then one track recorded and saved, output gpx and tcx files are not splitted on these track. It is good to have separate gpx file for each track.

@mru00
Copy link
Owner

mru00 commented Nov 10, 2014

Hello!

So my efforts found a user? Great! You are the first to contact me.

I already found my code in the AUR. Really surprised me. Good that my work helps others.

As you probably figured out the code base is really messy. I put it together in the evenings after my regular job- so the main goal was to patch it up quickly. Furthermore, it is written in perl. It was easy to do the reverse engineering and explore the protocol.

However, I decided to start a new repo with a proper client, without perl, without multiple scripts etc. I have implemented 95% of the functionality in a single c++ program, https://github.com/mru00/crane_gps_watch

Currently it only generates .Tcx files: .gpx files don't support tracks where only HR is available, but no gps coordinates- and I want to record my sessions on the home trainer.

To the topic: I would like to implement it! It was annoying me already. Today I think I found out how to distinguish sessions in a track, but could not investigate further. The watch sends an record with the 0x80 identifier, I guess this marks a stop/start.
If you can provide some info I would be glad to implement it, otherwise it will have to figure it out first.

Btw, which GUI are you using? I found "mytourbook" really good.

Rudolf Muehlbauer

On 10.11.2014, at 20:47, knedlyk notifications@github.com wrote:

It is not an issue, it is just a feature request. When Crane watch has more then one track recorded and saved, output gpx and tcx files are not splitted on these track. It is good to have separate gpx file for each track.


Reply to this email directly or view it on GitHub.

@knedlyk
Copy link
Author

knedlyk commented Nov 11, 2014

Hi Rudolf,
I have been using Gps watch from Conrad since the beginning of 2013. My attempts are not so good as yours, I didn't find a way how to decode data from the watch.

Today I think I found out how to distinguish sessions in a track, but could not investigate further. The watch sends an record with the 0x80 identifier, I guess this marks a stop/start.

Probably yes, need to check it. Another way is to split the tcx file by searching the


tags (end of previous activity and start of new one), the same is for gpx file:

Mytourbook is OK. I prefer to upload my gpx tracks to Strava, I found it is more suitable for me, it has statistics, segments, teams and so on...

Regardst,
Yarema

@mru00
Copy link
Owner

mru00 commented Nov 11, 2014

Hi Yarema,

I read your email wrong. Of course it is possible to split the tracks in single files, I will implement it as soon as I find the time.
But I would rather implement it in the "new" client https://github.com/mru00/crane_gps_watch . Did you have a look at it?

I guess it's ok to write files named after the start time of the track, and don't write files for tracks that already exist.

There is still the issue that I don't detect the stop/start again within a single track- this is what I referred to in your first mail.

Rudolf Muehlbauer

On 11.11.2014, at 09:06, knedlyk notifications@github.com wrote:

Yarema

@mru00
Copy link
Owner

mru00 commented Nov 11, 2014

Does strava work with tcx?
Then I could skip implementing gpx :)

But gpsbabel can convert tcx->gpx

Rudolf Muehlbauer

On 11.11.2014, at 09:06, knedlyk notifications@github.com wrote:

Hi Rudolf,
I have been using Gps watch from Conrad since the beginning of 2013. My attempts are not so good as yours, I didn't find a way how to decode data from the watch.

Today I think I found out how to distinguish sessions in a track, but could not investigate further. The watch sends an record with the 0x80 identifier, I guess this marks a stop/start.
Probably yes, need to check it. Another way is to split the tcx file by searching the

tags (end of previous activity and start of new one), the same is for gpx file:

Mytourbook is OK. I prefer to upload my gpx tracks to Strava, I found it is more suitable for me, it has statistics, segments, teams and so on...

Regardst,
Yarema


Reply to this email directly or view it on GitHub.

@knedlyk
Copy link
Author

knedlyk commented Nov 11, 2014

Yes, Strava works well both with tcx and gpx formats. I've just made a PKGBUILD for your new code https://aur.archlinux.org/packages/crane-gps-watch-git/ . I also had to add an option "-Wno-error=maybe-uninitialized" into CFLAGS beacuse of warning when compiling the code:
Watch.cpp:217:29: warning: ‘cur’ may be used uninitialized in this function [-Wmaybe-uninitialized]
parseWO(wo, first, 1+cur-first);

Regards,
Yarema

@mru00
Copy link
Owner

mru00 commented Nov 11, 2014

Oh that issue has to be fixed in the code.

Can you fork the repo? You can add your AUR files, patches... I will then merge your pull requests. I will also gladly accept documentation ;)

Rudolf Muehlbauer

On 11.11.2014, at 10:28, knedlyk notifications@github.com wrote:

Yes, Strava works well both with tcx and gpx formats. I've just made a PKGBUILD for your new code https://aur.archlinux.org/packages/crane-gps-watch-git/ . I also had to add an option "-Wno-error=maybe-uninitialized" into CFLAGS beacuse of warning when compiling the code:
Watch.cpp:217:29: warning: ‘cur’ may be used uninitialized in this function [-Wmaybe-uninitialized]
parseWO(wo, first, 1+cur-first);

Regards,
Yarema


Reply to this email directly or view it on GitHub.

@knedlyk
Copy link
Author

knedlyk commented Nov 11, 2014

Well, I can fork the repo, but please note that I'm not a real programmer,
I'm only interested in using this watch under linux 'cause I don't use
windows at all. My requirements are very easy: get track files from the
watch, and that's it. If you are interested in further development of your
client, take a look on this page:
https://www.runtastic.com/shop/en/runtastic-gps-watch-with-heart-rate-monitor
. It is a watch similar to Crane but with different software. Runtastic has
also the software named "Runtastic connect" (
http://www.runtastic.com/connect) which control all the functions of the
watch. Software is crappy and "heavyweight".

Yarema

2014-11-11 11:11 GMT+01:00 mru notifications@github.com:

Oh that issue has to be fixed in the code.

Can you fork the repo? You can add your AUR files, patches... I will then
merge your pull requests. I will also gladly accept documentation ;)

Rudolf Muehlbauer

On 11.11.2014, at 10:28, knedlyk notifications@github.com wrote:

Yes, Strava works well both with tcx and gpx formats. I've just made a
PKGBUILD for your new code
https://aur.archlinux.org/packages/crane-gps-watch-git/ . I also had to
add an option "-Wno-error=maybe-uninitialized" into CFLAGS beacuse of
warning when compiling the code:
Watch.cpp:217:29: warning: ‘cur’ may be used uninitialized in this
function [-Wmaybe-uninitialized]
parseWO(wo, first, 1+cur-first);

Regards,
Yarema


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub
#1 (comment).

@knedlyk
Copy link
Author

knedlyk commented Nov 11, 2014

Please add these two files (COPYING and README.md). It is a copy of GNU
public license, short description and version history. Please change
Readme.md for your needs as you wish.

Regards,
Yarema

2014-11-11 11:40 GMT+01:00 YUP yupadmin@gmail.com:

Well, I can fork the repo, but please note that I'm not a real programmer,
I'm only interested in using this watch under linux 'cause I don't use
windows at all. My requirements are very easy: get track files from the
watch, and that's it. If you are interested in further development of your
client, take a look on this page:
https://www.runtastic.com/shop/en/runtastic-gps-watch-with-heart-rate-monitor
. It is a watch similar to Crane but with different software. Runtastic has
also the software named "Runtastic connect" (
http://www.runtastic.com/connect) which control all the functions of the
watch. Software is crappy and "heavyweight".

Yarema

2014-11-11 11:11 GMT+01:00 mru notifications@github.com:

Oh that issue has to be fixed in the code.

Can you fork the repo? You can add your AUR files, patches... I will then
merge your pull requests. I will also gladly accept documentation ;)

Rudolf Muehlbauer

On 11.11.2014, at 10:28, knedlyk notifications@github.com wrote:

Yes, Strava works well both with tcx and gpx formats. I've just made a
PKGBUILD for your new code
https://aur.archlinux.org/packages/crane-gps-watch-git/ . I also had to
add an option "-Wno-error=maybe-uninitialized" into CFLAGS beacuse of
warning when compiling the code:
Watch.cpp:217:29: warning: ‘cur’ may be used uninitialized in this
function [-Wmaybe-uninitialized]
parseWO(wo, first, 1+cur-first);

Regards,
Yarema


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub
#1 (comment).

@knedlyk
Copy link
Author

knedlyk commented Nov 11, 2014

I forgot that I cannot attach files. Here they are:
COPYING: http://pastebin.com/download.php?i=2LwnR1p3
README.md: http://pastebin.com/download.php?i=k1WqifcL

Yarema

@mru00
Copy link
Owner

mru00 commented Nov 11, 2014

Pull request: Perfect, thank you!

On 2014-11-11 18:57, knedlyk wrote:

I forgot that I cannot attach files. Here they are:
COPYING: http://pastebin.com/download.php?i=2LwnR1p3 [1]
README.md: http://pastebin.com/download.php?i=k1WqifcL [2]

Yarema

Reply to this email directly or view it on GitHub [3].

Links:

[1] http://pastebin.com/download.php?i=2LwnR1p3
[2] http://pastebin.com/download.php?i=k1WqifcL
[3] #1 (comment)

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

No branches or pull requests

2 participants