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

Product (device name) not being recognized by parser #16

Open
MichaelLiss opened this issue May 1, 2020 · 13 comments
Open

Product (device name) not being recognized by parser #16

MichaelLiss opened this issue May 1, 2020 · 13 comments

Comments

@MichaelLiss
Copy link

Hi again,

I am successfully using the parser now ! (wow... such a learning curve on Java)

Anyway, I downloaded an activity, parsed it and got some results. However, I think the FR945 is not in the listing of products

I assume 'product' in the following output is going to show fr945 ??

"file_id": {
        "serial_number": 3996694396,
        "time_created": "2020-04-30T22:16:48.000Z",
        "manufacturer": "garmin",
        "product": 3113,
        "type": "activity"
    },

I see a 'product' like category in fit.js that contains an fr920xt, an edge 280 (both of which I own), the fr935 and fenix5x (both of which I know are watches)... but I don't see the fr945 ?

    garmin_product: {
      0: 'hrm_bike',
      1: 'hrm1',
      <snip>
      1765: 'fr920xt',
      <snip>
      2530: 'edge_820',
     <snip>
      2604: 'fenix5x',
      2606: 'vivo_fit_jr',
      2691: 'fr935',

I tried putting this code in and re-running the parser, but it did not pick it up:

      2859: 'descent',
      3113: 'fr945',
      10007: 'sdm4',
@MichaelLiss
Copy link
Author

MichaelLiss commented May 1, 2020

to verify that I am not going insane...

I changed this:

    manufacturer: {
      0: 0,
      1: 'garmin',

to this

    manufacturer: {
      0: 0,
      1: 'garmin_TEST',

ran the parser and got this, so I know I am not too crazy... LOL

"file_id": {
        "serial_number": 3996694396,
        "time_created": "2020-04-30T22:16:48.000Z",
        "manufacturer": "garmin_TEST",
        "product": 3113,
        "type": "activity"
    },

@jimmykane
Copy link
Owner

jimmykane commented May 1, 2020 via email

@MichaelLiss
Copy link
Author

MichaelLiss commented May 1, 2020

I changed the fit.js file... but it doesn't seem to be picking it up for some reason...

( the 'device_id' change did pick up btw )

      2859: 'descent',
      3113: 'fr945',
      10007: 'sdm4',

output:

    "file_id": {
        "device_id": 3996694396,
        "time_created": "2020-04-30T22:16:48.000Z",
        "manufacturer": "garmin",
        "product": 3113,
        "type": "activity"
    },

@MichaelLiss MichaelLiss changed the title forerunner 945 not listed as product Watches not being detected May 4, 2020
@MichaelLiss
Copy link
Author

I just ran another test with a fit file from a Garmin 920xt;

this is what I got:

  "file_id": {
    "serial_number": 3896976736,
    "time_created": "2020-05-04T20:17:04.000Z",
    "manufacturer": "garmin",
    "product": 1765,
    "type": "activity"
  }

This is what is in the fit.js file

      1752: 'hrm_run',
      1765: 'fr920xt',
      1821: 'edge510_asia',

@MichaelLiss MichaelLiss changed the title Watches not being detected Device name not being recognized by parser May 4, 2020
@MichaelLiss MichaelLiss changed the title Device name not being recognized by parser Product (device name) not being recognized by parser May 4, 2020
@jimmykane
Copy link
Owner

jimmykane commented May 5, 2020 via email

@MichaelLiss
Copy link
Author

MichaelLiss commented May 5, 2020

I can... I have one for a garmin 920xt and a 945.

The 920xt has actuall run data in it.
The 945 I just sat on my bed and clicked start / stop so there is no distance traveled.

here they are

FitFiles.zip

ps. All of the issues I wrote come from the same two devices (920xt and 945) fit files

@MichaelLiss
Copy link
Author

Did those help ?

@jimmykane
Copy link
Owner

jimmykane commented May 6, 2020 via email

@jimmykane
Copy link
Owner

Ok I see the issue here. Actually this was never implemented.

Meaning that the "matching" happens in the consumer lib. (so far)

So those ID are not used inside the lib.

Something could be done about that though.

@MichaelLiss
Copy link
Author

Can you update the code so it works with your component here ?

I don't know what 'consumer lib' means or how it ties into this component

Thanks !

@MichaelLiss
Copy link
Author

Hi....

I have been thinking about this.. and for backward compatibility (and stuff that doesn't have a name yet) what would be 'really' nice is to have both.

"file_id": {
        "device_id": 3996694396,
        "time_created": "2020-04-30T22:16:48.000Z",
        "manufacturer": "garmin",
        "product": 3113,
        "productName": "fr945",
        "type": "activity"
    },

in the case of no productName:

"file_id": {
        "device_id": 3996694396,
        "time_created": "2020-04-30T22:16:48.000Z",
        "manufacturer": "garmin",
        "product": 3113,
        "productName": "",
        "type": "activity"
    },

@MichaelLiss
Copy link
Author

Hello again...
I was wondering if you have some time to address this ?
I am getting to the point where this issue is starting to block the progress on my project.
(or maybe you could send some instructions on how to modify this so I can update it... and I will try to do the work???)

@MichaelLiss
Copy link
Author

Hi...

I am going to uninstall your module and use Garmin's parser and an AWS micro-service, as it doesn't seem you are going to work on this any time soon.

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