Skip to content

Update liblouis to 3.16.1#11888

Merged
michaelDCurran merged 4 commits into
nvaccess:betafrom
AAClause:liblouis-3.16
Dec 8, 2020
Merged

Update liblouis to 3.16.1#11888
michaelDCurran merged 4 commits into
nvaccess:betafrom
AAClause:liblouis-3.16

Conversation

@AAClause

@AAClause AAClause commented Dec 1, 2020

Copy link
Copy Markdown
Contributor

Link to issue number:

Fixes #11013
Fixes #11754

Summary of the issue:

Liblouis 3.16.0 has just been released.
EDIT: update to 3.16.1.

Description of how this pull request fixes the issue:

  • Updates the liblouis submodule (commit 48633b8).
  • Changes the readme.
  • Adds the following braille tables to the interface:
    • Bashkir grade 1
    • Russian grade 1
    • Russian grade 1 (detailed)
    • Coptic 8 dot computer
  • Removes the ru-ru-g1.utb entry

Testing performed:

Tested from source.

Known issues with pull request:

None

Change log entry:

@zstanecic

zstanecic commented Dec 1, 2020 via email

Copy link
Copy Markdown
Contributor

@LeonarddeR LeonarddeR closed this Dec 1, 2020
@LeonarddeR

Copy link
Copy Markdown
Collaborator

Ugh, I"m sorry for going so wild on my keyboard

@LeonarddeR LeonarddeR reopened this Dec 1, 2020
@LeonarddeR

Copy link
Copy Markdown
Collaborator

Sorry for being so brutal, but I think that in the future something needs to be done, so that NVDA can automatically discover liblouis tables. Now we have a problem in which we need to do this manually for every table, which is not so good.

I think you're not brutal at all but bringing up a fair point. There's an issue for that, see #11298
There is #290 as well, which is slightly related.

@LeonarddeR LeonarddeR left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

  1. Why is input disabled for the russian tables? I think it would be good to have a rationale in a comment.
  2. Given this release fixes two issues related to crashes and doesn't contain many additional changes to the core, I think we should consider shipping this with 2020.4. The beta period should be long enough to revert if it really breaks. cc @feerrenrut

@AAClause

AAClause commented Dec 1, 2020

Copy link
Copy Markdown
Contributor Author
  1. Why is input disabled for the russian tables? I think it would be good to have a rationale in a comment.

I simply used provided metadata to add these tables:

(0)$ cat ru-litbrl.ctb | grep "#+"
#+locale: ru
#+type: literary
#+dots: 6
#+contraction: no
#+direction: forward
(0)$ cat ru-litbrl-detailed.utb|grep "#+"
#+locale: ru
#+type: literary
#+dots: 6
#+contraction: no
#+direction: forward
#+variant: detailed
(0)$ 

Any thought @Futyn-Maker?

  1. Given this release fixes two issues related to crashes and doesn't contain many additional changes to the core, I think we should consider shipping this with 2020.4. The beta period should be long enough to revert if it really breaks.

👍

@Futyn-Maker

Futyn-Maker commented Dec 1, 2020

Copy link
Copy Markdown

Hello contributers! I was waiting for this PR. The couple of words about Russian tables:

  1. At the moment I'm not ready to develop the back translation according to rules of the Russian literary braille, it's quite difficult, so this table only for reading now, it's all right.
  2. I recommend to add all tables - ru-litbrl.ctb and ru-litbrl-detailed.utb to provide user's choice.

Andrey

@AAClause AAClause changed the title Update liblouis to 3.16.0 Update liblouis to 3.16.1 Dec 1, 2020
@Futyn-Maker

Futyn-Maker commented Dec 1, 2020

Copy link
Copy Markdown

I also would like to know if it is possible to replace "grade 1" with "literary braille" in the UI, or it can be done separate for the Russian localization. The reason is that we don't use contracted braille almost at all, so names "computer braille" and "literary braille" can be more understandable for end users than "grade 1", because there's no any gradation in fact. But it's not super important. @zstanecic do You have any comments about it?

@Futyn-Maker

Copy link
Copy Markdown

Thanks. As for me, looks good.

@LeonarddeR LeonarddeR added this to the 2020.4 milestone Dec 4, 2020
@LeonarddeR

Copy link
Copy Markdown
Collaborator

Added milestone 2020.4 for @feerrenrut or @michaelDCurran to triage.

LeonarddeR
LeonarddeR previously approved these changes Dec 4, 2020
@zstanecic

zstanecic commented Dec 4, 2020 via email

Copy link
Copy Markdown
Contributor

@AAClause

AAClause commented Dec 4, 2020

Copy link
Copy Markdown
Contributor Author

@zstanecic In my opinion, Leonard should have added this braille table in #11537.
Although I think it's out of scope for this PR, I am in favor of adding it. Should we make another pull request?

@lukaszgo1

Copy link
Copy Markdown
Contributor

Changelog entry for this one should probably mention the newly added tables.

@codeofdusk

Copy link
Copy Markdown
Contributor

I support merging this into 2020.4, so that the NVDA and Liblouis stable versions become once again in sync.

michaelDCurran
michaelDCurran previously approved these changes Dec 8, 2020
@feerrenrut

Copy link
Copy Markdown
Contributor

If we want this in 2020.4, it will need to be retargeted to beta

@AppVeyorBot

Copy link
Copy Markdown

See test results for failed build of commit f5fde1895e

@michaelDCurran michaelDCurran changed the base branch from master to beta December 8, 2020 03:49
@michaelDCurran michaelDCurran merged commit c7da025 into nvaccess:beta Dec 8, 2020
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.

Using English US computer Braille crashes NVDA when reading URLs NVDA often crashes when reading e-mail addresses

9 participants