move_coordinates supports generators (Sourcery refactored)

[sourcery-ai[bot]]

Pull Request #188 refactored by Sourcery.

Since the original Pull Request was opened as a fork in a contributor's repository, we are unable to create a Pull Request branching from it.

To incorporate these changes, you can either:

1. Merge this Pull Request instead of the original, or

2. Ask your contributor to locally incorporate these commits and push them to the original Pull Request

   Incorporate changes via command line

   git fetch https://github.com/cleder/pygeoif pull/188/head
   git merge --ff-only FETCH_HEAD
   git push

NOTE: As code is pushed to the original Pull Request, Sourcery will re-run and update (force-push) this Pull Request with new refactorings as necessary. If Sourcery finds no refactorings at any point, this Pull Request will be closed automatically.

See our documentation here.

Run Sourcery locally

Reduce the feedback loop during development by using the Sourcery editor plugin:

• VS Code
• PyCharm

Help us improve this pull request!

[watermelon-copilot-for-code-review[bot]]

WatermelonAI Summary

AI Summary deactivated by sourcery-ai[bot]

GitHub PRs

• #189 - move_coordinates supports generators (Sourcery refactored) - By sourcery-ai[bot] Just now

• #267 - Fix TimeStamp and TimeSpan class to use class_from_element method (Sourcery refactored) - By sourcery-ai[bot] 1 days ago

• #1 - Sourcery refactored main branch - By sourcery-ai[bot] 11 months ago

  No results found in Jira Tickets :(

  No results found in Confluence Docs :(

  No results found in Slack Threads :(

  No results found in Notion Pages :(

  No results found in Linear Tickets :(

  No results found in Asana Tasks :(

  pygeoif is an open repo and Watermelon will serve it for free. 🍉🫶 Why not invite more people to your team?

[pep8speaks]

Hello @sourcery-ai[bot]! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2023-11-12 22:08:56 UTC

[watermelon-copilot-for-code-review[bot]]

WatermelonAI Summary

AI Summary deactivated by sourcery-ai[bot]

GitHub PRs

• #189 - move_coordinates supports generators (Sourcery refactored) - By sourcery-ai[bot] Just now

• #10 - Update pygeoif to 1.1.0 - By pyup-bot 4 weeks ago

• #1 - Sourcery refactored main branch - By sourcery-ai[bot] 11 months ago

  No results found in Jira Tickets :(

  No results found in Confluence Docs :(

  No results found in Slack Threads :(

  No results found in Notion Pages :(

  No results found in Linear Tickets :(

  No results found in Asana Tasks :(

  pygeoif is an open repo and Watermelon will serve it for free. 🍉🫶

[what-the-diff[bot]]

PR Summary

• Improved Code Compatibility The development team added an import statement in pygeoif/functions.py, which means we now support more types of data without issues.
• New Test Cases New tests have been added to confirm the proper functioning of the move_coordinates function, ensuring the end result is reliable and error-free.
• Handling Data Generators A key improvement is the use of Generators in the move_coordinates function. This allows us to operate on very large datasets effectively and efficiently without running out of memory.
• Testing of Data Generators A test has been added to ensure move_coordinates produces a Generator, guaranteeing its correct implementation.
• Support for Different coordinate dimensions The final enhancement in this update includes test cases confirming that move_coordinates can handle coordinates of varying dimensions, expanding its usability across more diverse datasets.

[watermelon-copilot-for-code-review[bot]]

WatermelonAI Summary

AI Summary deactivated by sourcery-ai[bot]

GitHub PRs

• #189 - move_coordinates supports generators (Sourcery refactored) - By sourcery-ai[bot] Just now

• #267 - Fix TimeStamp and TimeSpan class to use class_from_element method (Sourcery refactored) - By sourcery-ai[bot] 1 days ago

• #1 - Sourcery refactored main branch - By sourcery-ai[bot] 11 months ago

  No results found in Jira Tickets :(

  No results found in Confluence Docs :(

  No results found in Slack Threads :(

  No results found in Notion Pages :(

  No results found in Linear Tickets :(

  No results found in Asana Tasks :(

  pygeoif is an open repo and Watermelon will serve it for free. 🍉🫶 Try us on any JetBrains IDE!

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ff2e39c) 100.00% compared to head (3f58c5e) 100.00%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## develop #189 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 21 21 Lines 2412 2430 +18 ========================================= + Hits 2412 2430 +18 ``` | [Files](https://app.codecov.io/gh/cleder/pygeoif/pull/189?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Christian+Ledermann) | Coverage Δ | | |---|---|---| | [pygeoif/functions.py](https://app.codecov.io/gh/cleder/pygeoif/pull/189?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Christian+Ledermann#diff-cHlnZW9pZi9mdW5jdGlvbnMucHk=) | `100.00% <100.00%> (ø)` | | | [tests/test\_functions.py](https://app.codecov.io/gh/cleder/pygeoif/pull/189?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Christian+Ledermann#diff-dGVzdHMvdGVzdF9mdW5jdGlvbnMucHk=) | `100.00% <100.00%> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[ghost]

👇 Click on the image for a new way to code review

#### [](https://app.codesee.io/r/reviews?pr=189&src=https%3A%2F%2Fgithub.com%2Fcleder%2Fpygeoif) #### Legend