Fix typo in argument names in `align_elements_using_landmarks()`
Open
Beginner friendly
Nobody has claimed this yet.
needs: triage
priority: low
- Dominant language
- Python
- Stars
- 394
- Forks
- 95
- Avg merge
- 4d 3h
- Merged PRs (30d)
- 7
Description
def align_elements_using_landmarks(
references_coords: Union[GeoDataFrame | DaskDataFrame],
should be reference_coords.
Not gonna fix now otherwise it will silently break the notebooks, as they are not in the CI yet.
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.
Research direction
Start at the definition of align_elements_using_landmarks() and search the repository notebooks and call sites for the plural argument name, since notebooks are not covered by CI. Correct the argument naming consistently and verify that the affected notebook usage still works; the issue does not name a specific test file.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 70/100