Headings at the start of chapters do not get relocated
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 45/100
Research direction
Start with VerseRef.ChangeVersification and trace how headings such as MAL 4:0/1:s1 are represented and converted. Then inspect UpdateUsfmParserHandler and the Serval pretranslation flow, using the Malachi example as the reproduction case. Done means headings at the start of a chapter remain in the generated USFM after versification changes.
Written by the indexing model from the issue text.
Description
I'm not sure this can be fixed.
Given the book Malachi with a source with the English Versification, and a target with the Septuagint Versification, most references will be converted correctly, e.g. MAL 4:1 -> MAL 3:19, and Serval will use these in the Pretranslations stored in Mongo. i.e.:
{
"textId": "MAL",
"sourceRefs": ["MAL 4:1"],
"targetRefs": ["MAL 3:19"],
"refs": ["MAL 3:19"],
"translation": "“For behold, the day is coming, burning like a furnace, when all the arrogant and every evildoer will be stubble; the day is coming when I will set them ablaze,” says the LORD of Hosts. “Not a root or branch will be left to them.”",
}
However, if there were headings at the start of chapter 4, VerseRef.ChangeVersification will not convert the source reference to a suitable target reference:
{
"textId": "MAL",
"sourceRefs": ["MAL 4:0/1:s1"],
"targetRefs": ["MAL 4:0/1:s1"],
"refs": ["MAL 4:0/1:s1"],
"translation": "The Day of the LORD",
}
Which means that the final USFM outputted by Serval using the UpdateUsfmParserHandler will lose the headings, i.e.:
\v 18 So you will again distinguish between the righteous and the wicked, between those who serve God and those who do not.”
\s1
\r
\m
\v 19 “For behold, the day is coming, burning like a furnace, when all the arrogant and every evildoer will be stubble; the day is coming when I will set them ablaze,” says the LORD of Hosts. “Not a root or branch will be left to them.”
I'm not sure what we can do? Check if the next verse is being relocated, and move any headings to the verse before that one???
- Dominant language
- C#
- Stars
- 29
- Forks
- 17
- Avg merge
- 4d 15h
- Merged PRs (30d)
- 8
Getting set up
We have not checked this project's setup files yet. Start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from sillsdev/machine
-
Difficulty 4/5 3-5 days Newbie friendliness 48/100
Maintainers usually reply within 1 day
-
Environment matchers pay for captures they never read; key the traversal on what the caller consumesOpen
Difficulty 5/5 Over a week Newbie friendliness 45/100
sillsdev/machine#515 · 1 comment ·
Maintainers usually reply within 1 day
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 35/100
Maintainers usually reply within 1 day
-
HermitCrab: Word.ExpandAlternatives discards Unify's success flag and can null the realizational FSOpen
Difficulty 5/5 Over a week Newbie friendliness 38/100
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 45/100
Maintainers usually reply within 1 day
All issues in sillsdev/machine
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
microsoft/fluentui-blazor#5364 ·
Maintainers usually reply within 1 day
-
.NET triage
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
microsoft/agent-framework#8811 ·
Maintainers usually reply within 1 day
-
.NET Docs
Difficulty 1/5 Under an hour Newbie friendliness 82/100
getsentry/sentry-dotnet#5637 · 1 comment ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
QuantConnect/Lean#9842 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
NethermindEth/nethermind#14012 ·
Maintainers usually reply within 1 day