SearchSpace tests should be refactored
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 55/100
Research direction
Start by reading test_searchspace.py, especially test_optim_forward_pass and the other repeated SearchSpace test cases. Refactor those cases to call a common function that accepts the SearchSpace object and relevant parameters. Done means the duplicated setup or assertions are consolidated and the SearchSpace tests still pass.
Written by the indexing model from the issue text.
Description
There's a lot of repeated code in the tests in test_searchspace.py, such as def test_optim_forward_pass. Refactor them so that they call a common test function which takes the SearchSpace object and other relevant parameters.
- Dominant language
- Python
- Stars
- 12
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
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.
More from automl/ConfigurableOptimizer
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
automl/ConfigurableOptimizer#252 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 45/100
automl/ConfigurableOptimizer#241 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
automl/ConfigurableOptimizer#207 ·
-
Fix Wandb crashOpen
Difficulty 4/5 3-5 days Newbie friendliness 35/100
automl/ConfigurableOptimizer#202 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 30/100
automl/ConfigurableOptimizer#201 ·
All issues in automl/ConfigurableOptimizer
Similar issues
-
Issue Triage-Needed
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
pulp/pulp_container#2581 ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 85/100
521xueweihan/HelloGitHub#4040 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 64/100
ClanGenOfficial/clangen#6313 ·
Maintainers usually reply within 2 days
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
Maintainers usually reply within 1 day
-
bug(trainer): CustomTrainer func_args with non literal values fail inside the TrainJobPossibly taken @avanishkasar claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
kubeflow/sdk#879 · 1 comment ·
Maintainers usually reply within 1 day