sillsdev / sillsdev/silnlp

Request for translate.py to error when requested model checkpoint is unavailable

Open
#593 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement needs to be revisited
Dominant language
Python
Stars
39
Forks
7
Avg merge
1d 9h
Merged PRs (30d)
5

Description

With the update to remove OpenNMT, silnlp no longer throws an error if the requested fine-tuned model checkpoint is unavailable. Instead, it outputs a well-formatted inference from the base model, which will contain garbage in most cases because the target language is not available in the base model. Because we now clean up models after just 30 days, this situation may arise more frequently. It would be helpful if translate.py could throw an error or warning when the requested model checkpoint is not found. Even having the ClearML job fail would be preferable to having it complete from the base model for an unknown language.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in translate.py and trace how the requested fine-tuned model checkpoint is selected when it is unavailable. Reproduce the missing-checkpoint case through the ClearML job, then verify that it reports an error or warning and does not complete inference with the base model.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
machine-learning
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.