A few small things found while updating Spanish

Đang mở
  • Có gì không ổn?
#961 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
50/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
rust, yaml
Lĩnh vực
accessibility, testing

Hướng nghiên cứu

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.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

  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.

Ngôn ngữ chính
Rust
Star
131
Fork
97
Merge trung bình
21 giờ 45 phút
Pull request đã merge (30 ngày)
95

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của daisy/MathCAT

Tất cả issue của daisy/MathCAT

Issue tương tự

Thêm issue về Rust

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.