`AntsArgParser` has conditional which is always `True`
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 85/100
Research direction
Start in lib/ants/command_parse.py at the command_parse entry point and line 92, then trace how target_obj is used when arguments are added. Confirm the conditional does not depend on a value that cannot change, and verify that command parsing still adds the expected arguments.
Written by the indexing model from the issue text.
Description
🐜 Bug Report
In command_parse there is a line declaring target_obj=self which creates a reference to self
Later arguments are added depending on if target_obj is self which always evaluates to True. It would be simpler to change the requirements of arguments to True.
How to reproduce
Step by step guide to reproduce the behaviour:
target_objassigned toselftarget_objis never re-assigned or copied in any execution path
Environment
I am using the ANTS 3.2.0 environment.
Version
This bug exists in ANTS 3.2.0/head of main
- Dominant language
- Python
- Stars
- 7
- Forks
- 10
- Avg merge
- 1d 19h
- Merged PRs (30d)
- 5
Contributor guide
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 MetOffice/ANTS
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
🐜 bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
📚 documentation
Difficulty 1/5 Under an hour Newbie friendliness 70/100
-
✨ enhancement
Difficulty 5/5 Over a week Newbie friendliness 25/100
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100