move_coordinates supports generators (Sourcery refactored)

[sourcery-ai]

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]

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! 🍻

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

[watermelon-copilot-for-code-review]

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]

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]

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]

Codecov Report

All modified and coverable lines are covered by tests ✅

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

Additional details and impacted files

@@            Coverage Diff            @@
##           develop      #189   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           21        21           
  Lines         2412      2430   +18     
=========================================
+ Hits          2412      2430   +18     

Files	Coverage Δ
pygeoif/functions.py	100.00% <100.00%> (ø)
tests/test_functions.py	100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ghost]

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

Legend