Skip to content

Add support for transductive alignment models#327

Open
ddaspit wants to merge 4 commits into
mainfrom
transductive-model
Open

Add support for transductive alignment models#327
ddaspit wants to merge 4 commits into
mainfrom
transductive-model

Conversation

@ddaspit

@ddaspit ddaspit commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

This change is Reviewable

ddaspit and others added 2 commits June 30, 2026 21:34
The Poetry 2.x relock tagged transformers/datasets (declared as both main
deps and extras) with `markers = extra == "huggingface"`, so CI's plain
`poetry install` no longer installed them and pyright failed on missing
imports. Relock with Poetry 1.8.2 (matching the repo's lock format) so the
sil-thot 3.5.1 bump is minimal and those packages stay unconditional.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@codecov-commenter

codecov-commenter commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.18321% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.97%. Comparing base (3520bdc) to head (3cd58ae).

Files with missing lines Patch % Lines
machine/translation/corpus_ops.py 85.71% 4 Missing ⚠️
...tion/thot/thot_symmetrized_word_alignment_model.py 94.44% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #327      +/-   ##
==========================================
+ Coverage   91.76%   91.97%   +0.20%     
==========================================
  Files         362      364       +2     
  Lines       23623    23739     +116     
==========================================
+ Hits        21678    21834     +156     
+ Misses       1945     1905      -40     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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