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

Detect Humanware Brailliant BI 40X and 20X via both USB and Bluetooth #11819

Merged
merged 2 commits into from
Dec 10, 2020

Conversation

michaelDCurran
Copy link
Member

Link to issue number:

None.

Summary of the issue:

Humanware BI 40X and 20X devices should be automatically detected via USB and Bluetooth.

Description of how this pull request fixes the issue:

Maps their respective USB IDs and Bluetooth names to our existing brailliantB driver.

Testing performed:

None - we don't have access to the devices, but we are informed these are compatible with the other Brailliant BI devices.

Known issues with pull request:

None.

Change log entry:

New features:

  • NVDA will automatically detect the Humanware BI 40X and 20X devices via both USB and Bluetooth.

LeonarddeR
LeonarddeR previously approved these changes Nov 9, 2020
Copy link
Collaborator

@LeonarddeR LeonarddeR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do these devices also have serial support like the previous generation?

@michaelDCurran
Copy link
Member Author

@LeonarddeR I don't know. I was not informed of that.

@LeonarddeR
Copy link
Collaborator

LeonarddeR commented Nov 9, 2020 via email

feerrenrut
feerrenrut previously approved these changes Nov 9, 2020
Copy link
Contributor

@feerrenrut feerrenrut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks fine to me. If we could get explicit confirmation from someone that this works before the release, that would be great.

@AppVeyorBot
Copy link

See test results for failed build of commit e212c4a0f3

@feerrenrut
Copy link
Contributor

I can see the error in the system test, silly off by one error. New PR coming soon.

@lukaszgo1
Copy link
Contributor

@michaelDCurran Diid you intent to merge this?

@LeonarddeR
Copy link
Collaborator

I think this should go in 2020.4 if possible.

@lukaszgo1
Copy link
Contributor

@LeonarddeR Mint adding this to the 2020.4 milestone to avoid this being forgotten before the release?

@michaelDCurran michaelDCurran added this to the 2020.4 milestone Dec 7, 2020
@michaelDCurran
Copy link
Member Author

There is some confusion remaining around whether these devices are released yet. I'm waiting on clarification from Humanware on what to do.

@feerrenrut
Copy link
Contributor

There is some confusion remaining around whether these devices are released yet. I'm waiting on clarification from Humanware on what to do.

Is there any danger to including this in 2020.4 even if the devices aren't released yet?

@michaelDCurran
Copy link
Member Author

michaelDCurran commented Dec 7, 2020 via email

@dpy013
Copy link
Contributor

dpy013 commented Dec 8, 2020

hi @michaelDCurran
This branch has conflicts that must be resolved
Only those with
to this repository can merge pull requests.
Conflicting files
user_docs/en/changes.t2t
thanks

@michaelDCurran
Copy link
Member Author

@feerrenrut I have retargetted this branch for beta now. Needs a review approval from you if you are happy. Humanware has confirmed this is oaky to go into NVDA 2020.4. Don't think they will provide us with any timely testing, nor we or anyone we know have access to one of those devices as they have not been released yet. However, the change is just adding extra mappings to more USB / Bluetooth IDs that use the same Humanware protocol.

Copy link
Contributor

@feerrenrut feerrenrut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks @michaelDCurran.

@michaelDCurran michaelDCurran merged commit 1b03387 into beta Dec 10, 2020
@michaelDCurran michaelDCurran deleted the brailliantBiX branch December 10, 2020 00:21
feerrenrut added a commit that referenced this pull request Dec 11, 2020
Merge remote-tracking branch 'origin/beta'
Including:
- Update translations.
- Update liblouis to 3.16.1 (PR #11888)
- Fix deadlock between nvWave stop and feed (PR #11886)
- Detect Humanware Brailliant BI 40X and 20X via both USB and Bluetooth (PR #11819)
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

Successfully merging this pull request may close these issues.

6 participants