Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: respect foreign match penalty in segmented matches #1231

Merged

Conversation

miurahr
Copy link
Member

@miurahr miurahr commented Dec 21, 2024

PR #1223 respect a penalty value of an external TM, but ignore a penalty of foreign language match

Which ticket is resolved?

  • BUGS#1251

What does this PR change?

  • Extent Scores class to hold penalty
  • Calculate maxPenalty with Scores.penalty
  • Enable Unit tests

Other information

  • Unit test of the case of foreign language match

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

@miurahr miurahr force-pushed the topoic/miurahr/matches/segmented-matches-expect-also-foreign branch from c39385e to b1ad4a5 Compare December 21, 2024 04:36
@miurahr miurahr merged commit c961d0e into master Dec 21, 2024
13 checks passed
@miurahr miurahr deleted the topoic/miurahr/matches/segmented-matches-expect-also-foreign branch December 21, 2024 05:10
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.

1 participant