PlaceMarkersUsfmUpdateBlockHandler gets confused with table cells
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Start with the UpdateUsfm_TableCells test and the PlaceMarkersUsfmUpdateBlockHandler it uses, and run the test to see the output. The bug is that table-cell markers such as \tc1 get moved to the end of the row and \tr rows lose their cell markers. Done means the test's target USFM matches the input USFM exactly, with the existing PlaceMarkers tests still passing.
索引モデルが issue の本文から書いたものです。
説明
The SF test team has noticed an issue where tables of a reasonably long length were getting garbled and out of order, both in normal builds and to a lesser degree (but still occurring) in echo builds. This particularly occurred with the SNHB DBL resource, which has a table in Numbers 1. The NLT does this too, but places verse markers in the table cells, which may alter the behavior of this bug.
The following test recreates the bug:
[Test]
public void UpdateUsfm_TableCells()
{
string source = "One Two Three Four Five Six Seven Eight Nine Ten Eleven Twelve Thirteen Fourteen Fifteen " +
"Sixteen Seventeen Eighteen Nineteen";
var alignInfo = new PlaceMarkersAlignmentInfo(
sourceTokens: [.. Tokenizer.Tokenize(source)],
translationTokens: [.. Tokenizer.Tokenize(source)],
alignment: ToWordAlignmentMatrix(
"0-0 1-1 2-2 3-3 4-4 5-5 6-6 7-7 8-8 9-9 10-10 11-11 12-12 13-13 14-14 15-15 16-16 17-17 18-18"
),
paragraphBehavior: UpdateUsfmMarkerBehavior.Preserve,
styleBehavior: UpdateUsfmMarkerBehavior.Strip
);
IReadOnlyList<UpdateUsfmRow> rows =
[
new UpdateUsfmRow(
ScrRef("MAT 1:1"),
source,
new Dictionary<string, object> { { "alignment_info", alignInfo } }
),
];
string usfm =
@"\id MAT
\c 1
\v 1 One
\tr \th1 Two \th2 Three
\tr \tc1 Four \tc2 Five
\tr \tc1 Six \tc2 Seven
\tr \tc1 Eight \tc2 Nine
\tr \tc1 Ten \tc2 Eleven
\tr \th1 Twelve \th2 Thirteen
\tr \tc1 Fourteen \tc2 Fifteen
\tr \tc1 Sixteen \tc2 Seventeen
\tr \tc1 Eighteen \tc2 Nineteen
";
string target = UpdateUsfm(
rows,
usfm,
paragraphBehavior: UpdateUsfmMarkerBehavior.Preserve,
usfmUpdateBlockHandlers: [new PlaceMarkersUsfmUpdateBlockHandler()]
);
AssertUsfmEquals(target, usfm);
}
The target USFM created by this test is:
\id MAT
\c 1
\v 1 One
\tr \th1 Two \th2 Three
\tr \tc1 Four \tc2 Five
\tr \tc1 Six \tc2 Seven
\tr \tc1 Eight \tc2 Nine \tc1
\tr Ten \tc2 Eleven
\tr \th1 Twelve \th2 Thirteen
\tr \tc1 Fourteen \tc2 Fifteen
\tr \tc1 Sixteen \tc2 Seventeen \tc1
\tr Eighteen \tc2 Nineteen
- 主要言語
- C#
- スター
- 29
- フォーク
- 16
- 平均マージ
- 6日 7時間
- マージ済み PR（30日）
- 14
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
sillsdev/machine のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
Hermit Crab
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
メンテナーはふだん 1 日以内に返信
-
porting
難易度 3/5 半日 初心者へのやさしさ 25/100
sillsdev/machine#617 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Hermit Crab
難易度 4/5 3〜5日 初心者へのやさしさ 18/100
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
メンテナーはふだん 1 日以内に返信
sillsdev/machine の issue をすべて見る
似ている issue
-
needs-area-label
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
メンテナーはふだん 2 日以内に返信
-
bug github_actions
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
endercodezz/BYDLO#336 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
SubtitleEdit/subtitleedit#16638 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
sebastienros/jint#4449 ·
メンテナーはふだん 1 日以内に返信