Skip to content

Improve Brazilian to European Portuguese contextual dialogue conversions - #358

Merged
niksedk merged 1 commit into
SubtitleEdit:mainfrom
SMAP-git:ptpt-contextual-dialogue-fixes
Nov 7, 2026
Merged

niksedk merged 1 commit into
SubtitleEdit:mainfrom
SMAP-git:ptpt-contextual-dialogue-fixes

Conversation

@SMAP-git

@SMAP-git SMAP-git commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Improve linked infinitive clitics, dropped-subject agreement, repeated predicates and reference-confirmed direct objects while retaining formal-register and third-person guards.
  • Recognize ellipsis vocatives and exclude third-person title appositions from formal-address propagation.
  • Add guarded natural dialogue, personal-name and family articles, beneficiary complements, medical arrest and emergency-response senses, demonstratives, slow-period adjectives and final interrogative accents.
  • Update both guides with neutral examples and standalone version 1.4.2. Preserve the upstream package version 1.5.0 for its release workflow.

Validation

  • Full tests against current upstream: 2,960 passed, 7 optional model tests skipped.
  • Self-contained Linux x64 package verified against 14 exact-output cases and a private 756-cue local corpus; all cue timings/counts preserved.
  • Added regression tests for options, ignored phrases, idempotence, conflicting meanings, formal address and split-cue interactions.
  • No private subtitle files or local corpus outputs are included.

Standalone Linux build: https://github.com/SMAP-git/BrazilianToEuropeanPortuguese/releases/tag/v1.4.2

Add guarded agreement, formal address, direct object, vocabulary sense and orthography corrections with bilingual documentation and regression tests. Keep upstream package version unchanged; document standalone 1.4.2.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@niksedk
niksedk merged commit 886ac9d into SubtitleEdit:main Nov 7, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants