Skip to content

build(deps): replace clipper2 with clipper2-rust#117

Merged
GreatV merged 3 commits into
mainfrom
clipper2
May 15, 2026
Merged

build(deps): replace clipper2 with clipper2-rust#117
GreatV merged 3 commits into
mainfrom
clipper2

Conversation

@GreatV
Copy link
Copy Markdown
Owner

@GreatV GreatV commented May 15, 2026

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request migrates the project from the clipper2 crate to clipper2-rust (v1.0.3). The changes include updating imports, transitioning from ClipperPath to PathD, and adapting to the new API for area calculation and path inflation. A suggestion was made to refactor the perimeter calculation loop to use a more idiomatic iterator-based approach and f64::hypot for improved numerical stability.

Comment thread oar-ocr-core/src/processors/db_bitmap.rs Outdated
@GreatV GreatV merged commit 8ba0ea1 into main May 15, 2026
7 checks passed
@GreatV GreatV deleted the clipper2 branch May 15, 2026 07:43
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.

1 participant