A few small things found while updating Spanish

Aperta
  • Qualcosa non va?
#961 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
50/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
rust, yaml
Ambito
accessibility, testing

Direzione di ricerca

Start with the proposed changes in Rules/Languages/en/overview.yaml and compare them with the Spanish rules. Then investigate the navigation errors and the Windows failure in src/speech.rs:3014, including the related tests and rule file mentioned in the report. The issue is done when the table dimensions are read correctly, the reported navigation cases work, and the test no longer hits the file access error.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

  1. f^{-1}(x) and f^{(4)}(x) are read as powers: "the negative 1 power of, f of x" and "the open paren 4 close paren power of, f of x". sin^{-1} is already read as inverse; for a function name followed by an argument, ^{-1} could be "f inverse of x" and ^{(n)} the n-th derivative.

  2. Overview of a plain mtable: a 2×3 table gives "the 3 by 0 table". overview.yaml (overview-default, tag mtable) uses count(*[2]/*) and count(*[2]/*[self::m:mtr][1]/*), which fit mrow with brackets; it should be count(*) and count(*[1]/*). Spanish already does this.

    Proposed patch (English; other languages have the same lines):

    diff --git a/Rules/Languages/en/overview.yaml b/Rules/Languages/en/overview.yaml
index 410ae12c..98df693f 100644
--- a/Rules/Languages/en/overview.yaml
+++ b/Rules/Languages/en/overview.yaml
@@ -88,9 +88,9 @@
   match: "."
   replace:
   - t: "the"
-  - x: count(*[2]/*)
+  - x: count(*)
   - t: "by"
-  - x: count(*[2]/*[self::m:mtr][1]/*)
+  - x: count(*[1]/*)
   - t: "table"
 
 - name: short-mrow

  3. Navigation errors:

    • DescribeCurrent inside a matrix (on a row or a cell) gives "Internal error: … can't DescribeCurrent from expression with id …".
    • <math><mi mathvariant="normal">$</mi><mn>10</mn></math>: ZoomIn and MoveNext give "Internal error: … can't ZoomIn/MoveNext from expression with id …".

  4. speech::tests::test_up_to_date sometimes fails on Windows (CI): "The process cannot access the file because it is being used by another process" (os error 32), at src/speech.rs:3014. It rewrites a zz rule file while another test may be reading it.

Lingua principale
Rust
Stelle
131
Fork
97
Merge medio
21h 45m
PR unite (30g)
95

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di daisy/MathCAT

Tutte le issue di daisy/MathCAT

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.