Skip to content

Conversation

@NiallOswald
Copy link
Contributor

Description

Closes #3

  • Replace visibility polygon algorithm
  • Re-enable tests for RDP

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

@mathleur mathleur self-requested a review October 28, 2025 13:40
@mathleur mathleur merged commit bc04be2 into ecmwf:main Oct 28, 2025
4 checks passed
@mathleur
Copy link
Member

Thanks for the nice fix @NiallOswald ! Will re-release v0.2.0 with the fixed algorithms :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Visibility algorithm in RDP fails for geometries with significant winding

2 participants