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

Bluetooth dc download from laptop not working. #1831

Closed
1 task done
cyberdiamond opened this issue Oct 27, 2018 · 10 comments
Closed
1 task done

Bluetooth dc download from laptop not working. #1831

cyberdiamond opened this issue Oct 27, 2018 · 10 comments

Comments

@cyberdiamond
Copy link
Contributor

cyberdiamond commented Oct 27, 2018

Describe the issue:

  • Bug

Issue long description:

Bluetooth dc download from laptop now not working.
Toshiba W50 laptop inbuilt Bluetooth now not picked up but used to work fine.

Operating system:

Windows 10 version 1809 build 17763.104

Subsurface version:

I noticed the new download box in some of the latest builds
Neither subsurface-4.8.3-263-ge0e3b6360100 or subsurface-4.8.3-271-g3f5f851fcb2d work.

Steps to reproduce:

Menu > Import > Import from dive computer
Check the Choose Bluetooth import download mode

Current behavior:

Down the bottom of the Bluetooth download window shows:
"The local Bluetooth adaptor cannot be accessed"

On the right hand side it shows:
Local Bluetooth device details
Name: Not available
Address: Not available

Expected behavior:

It should pickup/use the inbuilt Bluetooth radio.

Additional information:

I can still download over Bluetooth with multiple dive computers only if I create Bluetooth COM ports for each of them and use the old serial com port over Bluetooth mode.

Mentions:

Steve

@dirkhh
Copy link
Collaborator

dirkhh commented Oct 27, 2018

Bluetooth on Windows is full of challenges.
I haven't seen this one, yet. @neolit123 any ideas?

@neolit123
Copy link
Member

@cyberdiamond

please answer these questiosn:

if the above doesn't help share screenshots.

@cyberdiamond
Copy link
Contributor Author

SW Petrel 2 EXT, Ratio iX3m reb & HW OSTC3+

Yes Bluetooth enabled and working normally and downloads are working over the old serial COM port over Bluetooth method.

Device is paired in control panel.

Scan button is grey'd out see screenshot below.

bluetooth01

@neolit123
Copy link
Member

we seem to be using something that is not available on Windows:
http://doc.qt.io/qt-5/qbluetoothlocaldevice.html

On iOS and Windows, this class cannot be used because the platform does not expose any data or API which may provide information on the local Bluetooth device.

please try:
http://subsurface-divelog.org/downloads/test/subsurface-4.8.3-17-g53341c037d5a.exe

installers newer than that don't work for me either - scan button disabled.

@neolit123 neolit123 added the bug label Oct 28, 2018
@cyberdiamond
Copy link
Contributor Author

subsurface-4.8.3-17-g53341c037d5a.exe works properly.

bluetooth02

@neolit123
Copy link
Member

@dirkhh
after building latest Subsurface master, i've discovered that the scan button and control are enabled for me with my custom Qt version.
this on the other hand does not happen for our latest CI built Windows installers.

i cannot seem to figure out what the cause for that may be.

@dirkhh
Copy link
Collaborator

dirkhh commented Oct 28, 2018

The installers from the windows-container build should be using your custom Qt version. Or did I manage to break that? Crap.

@neolit123
Copy link
Member

if i install Subsurface from the latest CI installer and then temporary rename the Qt5Bluetooth.dll file so that it uses the DLL that i've built (by finding it in PATH), the scan button is enabled.

one difference between the two files is the size ~200kb.

@cyberdiamond
Copy link
Contributor Author

This is fixed in the latest builds

@dirkhh
Copy link
Collaborator

dirkhh commented Jun 24, 2019

Thanks for the update. MUCH appreciated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants