-
Notifications
You must be signed in to change notification settings - Fork 14
New prebuilt BLEServer.exe (x86 Release). #7
base: master
Are you sure you want to change the base?
Conversation
|
Thanks! Is that a DEBUG or RELEASE build? the file size has increased significantly |
|
Hi Uri,
That was an x86 Release build using Visual Studio 2017 Community Edition. The Debug build is actually much bigger.
Honestly, I would prefer for you to compile and update the BLEServer.exe for me rather than using my pull request. I just wasn't sure how to get a hold of you to ask you to do that. I am still very new to github.
I really appreciate that you accepted my change to allow Indicate to work. I'm not a BLE expert in the least, and I was very surprised to see that when I changed Notify to Indicate that my program suddenly started working. Up to that point I was only able to send Data to the BLE device. Lucky for me that I had 2 versions of the device and that I spotted the characteristic differences. I am very happy that you supported the feature to enumerate the characteristics supported.
Regards,
Brian
…________________________________
From: Uri Shaked <notifications@github.com>
Sent: Wednesday, May 22, 2019 7:25 AM
To: urish/noble-winrt
Cc: btetz14; Author
Subject: Re: [urish/noble-winrt] New prebuilt BLEServer.exe (x86 Release). (#7)
Thanks! Is that a DEBUG or RELEASE build? the file size has increased significantly
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#7?email_source=notifications&email_token=AMDHW6F6QBNS3TTWMQN3VRLPWTYP5A5CNFSM4HOE7R42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV6E5XY#issuecomment-494685919>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AMDHW6EC7ICZ3KEMYD6GSHDPWTYP5ANCNFSM4HOE7R4Q>.
|
|
Actually, I'd love if we could get it to build automatically on Travis. If
you have some time to look into it, it seems that they provide a Windows
environment with Visual Studio preinstalled:
https://blog.travis-ci.com/2018-10-11-windows-early-release
…On Thu, May 23, 2019, 1:27 PM btetz14 ***@***.***> wrote:
Hi Uri,
That was an x86 Release build using Visual Studio 2017 Community Edition.
The Debug build is actually much bigger.
Honestly, I would prefer for you to compile and update the BLEServer.exe
for me rather than using my pull request. I just wasn't sure how to get a
hold of you to ask you to do that. I am still very new to github.
I really appreciate that you accepted my change to allow Indicate to work.
I'm not a BLE expert in the least, and I was very surprised to see that
when I changed Notify to Indicate that my program suddenly started working.
Up to that point I was only able to send Data to the BLE device. Lucky for
me that I had 2 versions of the device and that I spotted the
characteristic differences. I am very happy that you supported the feature
to enumerate the characteristics supported.
Regards,
Brian
________________________________
From: Uri Shaked ***@***.***>
Sent: Wednesday, May 22, 2019 7:25 AM
To: urish/noble-winrt
Cc: btetz14; Author
Subject: Re: [urish/noble-winrt] New prebuilt BLEServer.exe (x86 Release).
(#7)
Thanks! Is that a DEBUG or RELEASE build? the file size has increased
significantly
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
#7?email_source=notifications&email_token=AMDHW6F6QBNS3TTWMQN3VRLPWTYP5A5CNFSM4HOE7R42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV6E5XY#issuecomment-494685919>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/AMDHW6EC7ICZ3KEMYD6GSHDPWTYP5ANCNFSM4HOE7R4Q
>.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#7?email_source=notifications&email_token=AAGZ3HR47NC6DAXYABCVS6TPWZWSHA5CNFSM4HOE7R42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWBZISI#issuecomment-495162441>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAGZ3HU57GE3IOEEGRIOMELPWZWSHANCNFSM4HOE7R4Q>
.
|
|
I'm really, sorry. I don't know what Travis is. I'm a bare metal programmer, who is a little out of his element in the npm world. I have a team helping me build the GUI app out of typescript, electron, and angular. To be honest, I didn't even think I would be able to get the BLE work natively from Windows 10. The first project that I tried, noble-uwp, did not install and run properly in my project, and I was about to give up. When I tried your project, I saw a glimmer of hope.
Do you think it would be possible to convert your server to compile with npm-gyp as a native c++ addons?
Regards,
Brian
________________________________
From: Uri Shaked <notifications@github.com>
Sent: Thursday, May 23, 2019 8:08 PM
To: urish/noble-winrt
Cc: btetz14; Author
Subject: Re: [urish/noble-winrt] New prebuilt BLEServer.exe (x86 Release). (#7)
Actually, I'd love if we could get it to build automatically on Travis. If
you have some time to look into it, it seems that they provide a Windows
environment with Visual Studio preinstalled:
https://blog.travis-ci.com/2018-10-11-windows-early-release
On Thu, May 23, 2019, 1:27 PM btetz14 ***@***.***> wrote:
Hi Uri,
That was an x86 Release build using Visual Studio 2017 Community Edition.
The Debug build is actually much bigger.
Honestly, I would prefer for you to compile and update the BLEServer.exe
for me rather than using my pull request. I just wasn't sure how to get a
hold of you to ask you to do that. I am still very new to github.
I really appreciate that you accepted my change to allow Indicate to work.
I'm not a BLE expert in the least, and I was very surprised to see that
when I changed Notify to Indicate that my program suddenly started working.
Up to that point I was only able to send Data to the BLE device. Lucky for
me that I had 2 versions of the device and that I spotted the
characteristic differences. I am very happy that you supported the feature
to enumerate the characteristics supported.
Regards,
Brian
________________________________
From: Uri Shaked ***@***.***>
Sent: Wednesday, May 22, 2019 7:25 AM
To: urish/noble-winrt
Cc: btetz14; Author
Subject: Re: [urish/noble-winrt] New prebuilt BLEServer.exe (x86 Release).
(#7)
Thanks! Is that a DEBUG or RELEASE build? the file size has increased
significantly
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
#7?email_source=notifications&email_token=AMDHW6F6QBNS3TTWMQN3VRLPWTYP5A5CNFSM4HOE7R42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV6E5XY#issuecomment-494685919>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/AMDHW6EC7ICZ3KEMYD6GSHDPWTYP5ANCNFSM4HOE7R4Q
>.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#7?email_source=notifications&email_token=AAGZ3HR47NC6DAXYABCVS6TPWZWSHA5CNFSM4HOE7R42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWBZISI#issuecomment-495162441>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAGZ3HU57GE3IOEEGRIOMELPWZWSHANCNFSM4HOE7R4Q>
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#7?email_source=notifications&email_token=AMDHW6B6YCKF2DTPIUKTUQ3PW32VHA5CNFSM4HOE7R42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWDLCGQ#issuecomment-495366426>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AMDHW6F6VBZC7ISHSWMV7UTPW32VHANCNFSM4HOE7R4Q>.
|
|
I was looking closer at your web-bluetooth-polyfill project and it says "Depracted - Use Chrome Canary instead".
Does this mean that the correct way to support native BLE in Windows 10 is to use https://github.com/thegecko/webbluetooth?
My project is running Angular in Electron and I was looking for a way to support using the Native BLE driver support in windows 10, instead of using a hacked driver like the Nobel project suggest. I was searching for a Node project that I could use, and that is how I came across your project, Nobel-winrt. As long as I patch the ble-server, it works. It has its quirks, like it does not seem to look at the status of the Bluetooth enable, but it is working. There are a few other things I noticed, like if I disconnect, and reconnect, the list of devices grows. It does not clear out the previously found, or check the list to see if the previously found was already in there. My solution to that was to just not disconnect and reconnect.
Anyways, my question is, do you know if webbluetooth would be able to do the same thing as nobel-winrt, and if so, is that the correct way to go about doing what I want to do?
Thanks,
Brian
________________________________
From: Uri Shaked <notifications@github.com>
Sent: Thursday, May 23, 2019 8:08 PM
To: urish/noble-winrt
Cc: btetz14; Author
Subject: Re: [urish/noble-winrt] New prebuilt BLEServer.exe (x86 Release). (#7)
Actually, I'd love if we could get it to build automatically on Travis. If
you have some time to look into it, it seems that they provide a Windows
environment with Visual Studio preinstalled:
https://blog.travis-ci.com/2018-10-11-windows-early-release
On Thu, May 23, 2019, 1:27 PM btetz14 ***@***.***> wrote:
Hi Uri,
That was an x86 Release build using Visual Studio 2017 Community Edition.
The Debug build is actually much bigger.
Honestly, I would prefer for you to compile and update the BLEServer.exe
for me rather than using my pull request. I just wasn't sure how to get a
hold of you to ask you to do that. I am still very new to github.
I really appreciate that you accepted my change to allow Indicate to work.
I'm not a BLE expert in the least, and I was very surprised to see that
when I changed Notify to Indicate that my program suddenly started working.
Up to that point I was only able to send Data to the BLE device. Lucky for
me that I had 2 versions of the device and that I spotted the
characteristic differences. I am very happy that you supported the feature
to enumerate the characteristics supported.
Regards,
Brian
________________________________
From: Uri Shaked ***@***.***>
Sent: Wednesday, May 22, 2019 7:25 AM
To: urish/noble-winrt
Cc: btetz14; Author
Subject: Re: [urish/noble-winrt] New prebuilt BLEServer.exe (x86 Release).
(#7)
Thanks! Is that a DEBUG or RELEASE build? the file size has increased
significantly
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
#7?email_source=notifications&email_token=AMDHW6F6QBNS3TTWMQN3VRLPWTYP5A5CNFSM4HOE7R42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV6E5XY#issuecomment-494685919>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/AMDHW6EC7ICZ3KEMYD6GSHDPWTYP5ANCNFSM4HOE7R4Q
>.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#7?email_source=notifications&email_token=AAGZ3HR47NC6DAXYABCVS6TPWZWSHA5CNFSM4HOE7R42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWBZISI#issuecomment-495162441>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAGZ3HU57GE3IOEEGRIOMELPWZWSHANCNFSM4HOE7R4Q>
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#7?email_source=notifications&email_token=AMDHW6B6YCKF2DTPIUKTUQ3PW32VHA5CNFSM4HOE7R42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWDLCGQ#issuecomment-495366426>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AMDHW6F6VBZC7ISHSWMV7UTPW32VHANCNFSM4HOE7R4Q>.
|
|
Hi Brain, Web Bluetooth can do a subset of what noble (and thus also noble-winrt) can do. For instance, you are not able to connect to a device without the user explicitly choosing this device from on every single invocation of the application. So it really depends on your use case. In addition, it is not enable by default on Linux, AFAIK. I guess copying the server code to this repo (so it lives together with the code that uses it) and compiling it using node-gyp with bindings would be the best way to go, but I don't have the time to do this. If you are up to it, I can try to give some guidance. |
|
I like your idea! I am definitely willing to give it a try. I will have time to start working on that from next week.
To be sure, your idea is to turn it into a c++ addon, right? I've written a c++ addon once. I was able to load a c++ dll and call it from node, but all of that was just a test to prove out an idea.
Brian
…________________________________
From: Uri Shaked <notifications@github.com>
Sent: Friday, May 31, 2019 6:29 AM
To: urish/noble-winrt
Cc: btetz14; Author
Subject: Re: [urish/noble-winrt] New prebuilt BLEServer.exe (x86 Release). (#7)
Hi Brain,
Web Bluetooth can do a subset of what noble (and thus also noble-winrt) can do. For instance, you are not able to connect to a device without the user explicitly choosing this device from on every single invocation of the application. So it really depends on your use case. In addition, it is not enable by default on Linux, AFAIK.
I guess copying the server code to this repo (so it lives together with the code that uses it) and compiling it using node-gyp with bindings would be the best way to go, but I don't have the time to do this. If you are up to it, I can try to give some guidance.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#7?email_source=notifications&email_token=AMDHW6BS6OHJKN6WE5IKXG3PYDAT5A5CNFSM4HOE7R42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWUKQ3Q#issuecomment-497592430>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AMDHW6AZZ5HUVAUGSHRTNH3PYDAT5ANCNFSM4HOE7R4Q>.
|
|
Yes, exactly! |
|
Is this ever going to be merged? |
|
Not really, see the discussion above. However, you are invited to fork and maintain your own repo with the new version, if you like |
|
So the discussion was about merging the source for BLEServer.exe instead of updating it periodically. Got it. The noble ecosystem is terribly fractured at the moment. Is there a way that the bindings from noble-winrt could be extracted, and then used with the users preferred for of noble (https://github.com/abandonware/noble for example)? It looks like the original noble had a method/module for running with custom bindings.
|
|
Sounds like it could work... Unfortunately, I don't have bandwidth for this project anymore, so feel free to explore, fork, etc. |
Built with VS2017 Community Edition. This new prebuilt binary includes the recent Indication subscribe fix.