WUR-AI / WUR-AI/diffWOFOST

[Task] Replace EngineTestHelper with Engine

Open
#90 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
25
Forks
6
Avg merge
2d 23h
Merged PRs (30d)
1

Description

After #25, we have to replace EngineTestHelper with Engine. Apply changes in:

  • crop modules
  • notebooks
  • tests
  • dont fix import from parameter_provider in notebook, check if related fix has been merged in pcse, see #124
  • Also fix changes introduced by #125 in all notebooks, import from utils should be replaced by import from test

Contributor guide

Open the contributing guide

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 by reviewing the changes from #25, then inspect the crop modules, notebooks, and tests for EngineTestHelper references. Check #124 for the parameter_provider import and #125 for notebook imports from utils. Done means the affected modules and notebooks use Engine consistently and the listed import changes are resolved.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
developer-experience, testing-qa
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.