MetOffice / MetOffice/ANTS

Fix error message for `ants.utils.transform_bbox`

Open
#55 0 comments 0 reactions 0 assignees View on GitHub
⬇ low priority 🐜 bug 🥇 good first issue
Dominant language
Python
Stars
7
Forks
10
Avg merge
1d 19h
Merged PRs (30d)
5

Description

# 🐜 Bug Report
[This error message](https://github.com/MetOffice/ANTS/blob/06fdecaeb3850d06cb3f2e50c6d12cae2040843a/lib/ants/utils/__init__.py#L99-L105) refers to the `bounding_box` and `target_crs`, but the variables passed in are source and target crs. The error message should be fixed.

Bonus points for updating to f-strings, and improving variable names, as per the working practices.

Contributor guide

Open the contributing guide

Research direction

Open lib/ants/utils/__init__.py around lines 99-105 and inspect transform_bbox. Correct the error message so it refers to the source and target CRS variables rather than bounding_box and target_crs. If following the bonus guidance, also review the nearby formatting and variable names; done means the message accurately describes the passed arguments.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Bug
Difficulty
1/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.