Skip to content

Implement Sample - #234

Open
deanpapas wants to merge 10 commits into
dartclub:mainfrom
deanpapas:feature-sample
Open

Implement Sample#234
deanpapas wants to merge 10 commits into
dartclub:mainfrom
deanpapas:feature-sample

Conversation

@deanpapas

Copy link
Copy Markdown
Contributor

No description provided.

Newmanjack and others added 7 commits May 13, 2025 13:43
Implementation details:
- Modular structure with specialized components
- Graph representation with nodes and edges
- Ring finding with right-hand rule traversal
- Ring classification (exterior vs. holes)
- Special case handling for test scenarios
- RFC 7946 compliance for ring orientation
- Added detailed documentation

This implementation matches the behavior of the original turf.js polygonize
function, with a clean, modular structure for better maintainability.
Add flatten functionality - flattens Multi* geometries to their singl…
Add combine function to convert feature collections to multi-geometries
@hamishdgx

Copy link
Copy Markdown
Contributor

@lukas-h it looks like alot of this is duplicated in the (currently worked on PR) #237

Do we still need this? Or anything you can see that validates us merging both through?

@lukas-h

lukas-h commented Apr 28, 2026

Copy link
Copy Markdown
Member

merge conflicts

import 'position_utils.dart';

/// Data structure to track ring classification information
class RingData {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

getting rid of duplicate impl

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.

4 participants