Skip to content

Commit b321356

Browse files
Reach the wrong-case apposition fallback
The fallback that lets an apposition or parenthesis in the wrong case parse with an error tag ("Ég ræddi við forsætisráðherra (Sigmundur Davíð)") was defined under the misspelt name Svigainnihald_* and thus never referenced, although GreynirCorrect still has the handler for its AðvörunSvigaInnihaldNl node. Attach it as the last, lowest-priority alternative of SvigaInnihald via a per-case helper. A separate definition block would have gotten top priority and pruned the correct readings, since the priority counter of a block is not shared with the generic definition. On the 1000-sentence baseline 2 trees change, both improved (a parenthesis "( átta )" now attaches as an apposition instead of forcing a heading analysis). Engine and GreynirCorrect suites pass. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent d4c4576 commit b321356

1 file changed

Lines changed: 9 additions & 6 deletions

File tree

src/reynir/Greynir.grammar

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5459,12 +5459,15 @@ SvigaInnihald/tala/fall/kyn →
54595459
> Nl/fall # Sama fall en ekki sama tala
54605460
> SvigaInnihaldFsRuna
54615461
> HreinYfirsetning
5462-
5463-
# Ef ekki finnst sama fall og tala, prófa almennan nafnlið
5464-
Svigainnihald_nf/tala/pers/kyn → AðvörunSvigaInnihaldNl/fallxnf
5465-
Svigainnihald_þf/tala/pers/kyn → AðvörunSvigaInnihaldNl/fallxþf
5466-
Svigainnihald_þgf/tala/pers/kyn → AðvörunSvigaInnihaldNl/fallxþgf
5467-
Svigainnihald_ef/tala/pers/kyn → AðvörunSvigaInnihaldNl/fallxef
5462+
# Ef ekki finnst sama fall og tala, prófa nafnlið í öðru falli (villa).
5463+
# Þetta verður að vera aftast í forgangsröðinni hér, þar sem sérstök
5464+
# skilgreining (SvigaInnihald_nf/tala/kyn → ...) fengi hæsta forgang.
5465+
> SvigaInnihaldAnnaðFall/fall
5466+
5467+
SvigaInnihaldAnnaðFall_nf → AðvörunSvigaInnihaldNl/fallxnf
5468+
SvigaInnihaldAnnaðFall_þf → AðvörunSvigaInnihaldNl/fallxþf
5469+
SvigaInnihaldAnnaðFall_þgf → AðvörunSvigaInnihaldNl/fallxþgf
5470+
SvigaInnihaldAnnaðFall_ef → AðvörunSvigaInnihaldNl/fallxef
54685471

54695472
# Árabil nota en-dash
54705473

0 commit comments

Comments
 (0)