Create minimal test suite for common scripts in SILNLP
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 39
- Forks
- 7
- Avg merge
- 1d 9h
- Merged PRs (30d)
- 5
Description
There are currently a variety of different scripts in SILNLP that are commonly used. In order to reduce the likelihood that a change to the repo results in one of these scripts breaking, we should have a folder with a basic test for each of the scripts that could be run before merging a PR. The scripts to cover should include:
silnlp.nmt.preprocess
silnlp.nmt.train
silnlp.nmt.test
silnlp.nmt.translate (we can use silnlp.nmt.experiment for all four of these nmt scripts)
silnlp.smt.preprocess
silnlp.smt.train
silnlp.smt.test
silnlp.common.analyze
silnlp.common.extract_corpora
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the silnlp.nmt, silnlp.smt, and silnlp.common script entry points named in the issue, and review how silnlp.nmt.experiment can cover the NMT scripts. Add a folder containing a basic test for each listed script, then run the complete test suite to verify these common scripts can be checked before a pull request is merged.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- testing-qa
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100