Updated company_ids.json

[dinesharjani]

No description provided.

[eriklins]

There is an empty entry 3619, which - I think - should be removed. It's also missing in the original BT SiG YAML file: https://bitbucket.org/bluetooth-SIG/public/src/main/assigned_numbers/company_identifiers/company_identifiers.yaml (0x0E23 has not been assigned, not sure why).

[dinesharjani]

There is an empty entry 3619, which - I think - should be removed. It's also missing in the original BT SiG YAML file: https://bitbucket.org/bluetooth-SIG/public/src/main/assigned_numbers/company_identifiers/company_identifiers.yaml (0x0E23 has not been assigned, not sure why).

What a keen eye! I just... yeah, I added it because I don't like the gaps being there. But I guess you're right.

[eriklins]

Looks good now from my perspective.

[dinesharjani]

Hey @eriklins - I'm going to keep 3619 as "Reserved". There's already the precedent of one of these missing IDs to get assigned later. I think it makes sense to keep it. It might never be used, but we can decide later if we keep getting these skips in the Company ID list and modify the scripts to accommodate them.

[eriklins]

Yeah, now as you mention that, I had added this to the Python script I sent to you a while back. It actually inserts a "Reserved" entry on missing entries.

[dinesharjani]

Yeah, now as you mention that, I had added this to the Python script I sent to you a while back. It actually inserts a "Reserved" entry on missing entries.

Oh. Okay. Going to look at that now, since I'm here.

[dinesharjani]

Okay, so. I see your script here https://github.com/eriklins/bt-assigned-numbers-to-c-header/blob/main/bt_assigned_numbers_to_c.py

My only issue with it, is that it's Python. Now, I'm not a "web developer". This project is in Node.js because when I built it, the other developers at Nordic that lent me a hand suggested Node.js. Since most of this is node.js, and the Github actions are set up to run node.js, I think the best way to keep this project going forward is to keep it in Node.js / JavaScript. Not sure if I want to tackle writing the whole thing in Javascript now, though I could try feeding it to some ... "artificial intellgence". But I just wanted to come back to you and tell you where my head's at.

[eriklins]

No, this actually is a different script. This one creates C header file from the JSONs here. I sent the one to convert the BT SiG YAML to company_ids.json a while back somewhere here in one of our conversations. Anyway, I arched it again here.

I had never worked with GitHub actions and my experience with JS is quite limited. I used Python for a bunch of scripting stuff and hence used it for the YAML to JSON converter. I'm happy to run this on a regular basis and do a pull request here but I'm not going to port it to Javascript, I'm afraid.

company_ids_to_json.zip