Skip to content

Handle all Missing entities and migrate from System.Drawing to Scia#29

Open
rodion-m wants to merge 16 commits intoDomCR:masterfrom
rodion-m:master
Open

Handle all Missing entities and migrate from System.Drawing to Scia#29
rodion-m wants to merge 16 commits intoDomCR:masterfrom
rodion-m:master

Conversation

@rodion-m
Copy link

No description provided.

rodion-m and others added 16 commits February 17, 2026 12:47
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add IR primitives, property resolver, flattener, and PDF backend behind UseSceneGraph flag; include unit tests.
Replace System.Drawing-based SVG rasterization with Svg.Skia + SkiaSharp and update preview test assertions.
Python/ezdxf script generates 10 targeted DXF fixture files (stages 00-09).
C# XUnit integration tests load each fixture and export via both legacy and
scene-graph pipelines, verifying entity counts and valid PDF output.
Stages 05-09 export tests are skipped until rendering is implemented.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add Clipper2-based hatch pattern clipping, solid/pattern/gradient handling, and extensive unit + integration coverage.
@rodion-m
Copy link
Author

Hi @DomCR ! Coud you please help with a proper verification that rendering of all these entities works correct? It was implemented, revied and tested by Codex powered GPT 5.2 high/xhigh. So, we should properly verify the outcome.
Also, it was neccessery to improve the code ACadSharp library, is it ok if I open a separate PR in the main repo for these changes?

@DomCR
Copy link
Owner

DomCR commented Feb 21, 2026

Hi @rodion-m,

The PR doesn't seem to focus on a single feature and it changes a lot of code in the core, I would suggest to split it into smaller PRs so it can be tested and reviewed properly.

The PR contains a lot of dxf that are generated by ezdxf, they can be ignored due that they will be generated before testing.
I don't understand why do we need to add ezdxf to create dxf, I would prefer to test ACadSharp itself to generate the files.

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.

2 participants