Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DXF: add the support of AutoCAD Wipeout object #11022

Open
Jwohnlf opened this issue Oct 15, 2024 · 3 comments · May be fixed by #11720
Open

DXF: add the support of AutoCAD Wipeout object #11022

Jwohnlf opened this issue Oct 15, 2024 · 3 comments · May be fixed by #11720
Labels
enhancement

Comments

@Jwohnlf
Copy link

Jwohnlf commented Oct 15, 2024 •
edited
Loading

Feature description

I open this issue with the intention to add an feature enhancement to add the support of WIPEOUT DXF entities.
As mentionned in the knonw issue log, this feature is still missing in the DXF driver.

WIPEOUT objects are sometimes used to draw the outline of object. In the screenshot below a annotation box related to a sewer manhole is drawn as a block using such DXF entity.
DXF-wipeout-instance1

Another example where a manhole outline is drawn with a Wipeout entity
DXF-wipeout-instance2

Additional context

No response
@rouault
Copy link
Member

rouault commented Oct 15, 2024

FYI @atlight

@atlight
Copy link
Contributor

atlight commented Oct 16, 2024

Actually making the WIPEOUT entity perform the wipeout action on other geometries may be beyond the scope of the GDAL DXF driver, but it surely wouldn't be too difficult to construct the outline of the WIPEOUT as a polygon so that the consumer can do what it wants with the raw geometry.

In any event I have little or no bandwidth for coding feature requests on the DXF driver at this time. I'd be happy to review pull requests though.

@Jwohnlf
Copy link
Author

Jwohnlf commented Oct 17, 2024

My intention is indeed to propose a feature implementation limited to the output the outline of these WIPEOUT entities.
I'll do a pull request soon. Your review is obviously welcome !

Jwohnlf added a commit to Jwohnlf/gdal that referenced this issue Nov 9, 2024
@Jwohnlf
DXF: add Wipeout translate function (fixes OSGeo#11022)
0f6eeb3
Jwohnlf added a commit to Jwohnlf/gdal that referenced this issue Dec 18, 2024
@Jwohnlf
DXF: amend the log of known issues (fixes OSGeo#11022)
3490760
Jwohnlf added a commit to Jwohnlf/gdal that referenced this issue Dec 19, 2024
@Jwohnlf
DXF: amend the log of known issues (fixes OSGeo#11022)
fd7b2e8
Jwohnlf added a commit to Jwohnlf/gdal that referenced this issue Dec 19, 2024
@Jwohnlf
DXF: mention new WIPEOUT support into doc (fixes OSGeo#11022)
eb89d2d
Jwohnlf added a commit to Jwohnlf/gdal that referenced this issue Jan 25, 2025
@Jwohnlf
DXF: Unit test for wipeout geometry (fixes OSGeo#11022)
a07fb61
Jwohnlf added a commit to Jwohnlf/gdal that referenced this issue Jan 26, 2025
@Jwohnlf
DXF: add unit test for wipeout parsing (fixes OSGeo#11022)
e5a1518
Jwohnlf added a commit to Jwohnlf/gdal that referenced this issue Jan 26, 2025
@Jwohnlf
DXF: add unit test for wipeout parsing (fixes OSGeo#11022)
5af196f
Jwohnlf added a commit to Jwohnlf/gdal that referenced this issue Jan 26, 2025
@Jwohnlf
DXF: add unit test for wipeout parsing (fixes OSGeo#11022)
126e525
@Jwohnlf Jwohnlf linked a pull request Jan 26, 2025 that will close this issue
Open
Jwohnlf added a commit to Jwohnlf/gdal that referenced this issue Feb 6, 2025
@Jwohnlf
DXF: skip WIPEOUT with unsupported point format (fixes OSGeo#11022)
c236356
Jwohnlf added a commit to Jwohnlf/gdal that referenced this issue Feb 16, 2025
@Jwohnlf
DXF: downsize unit test file (fixes OSGeo#11022)
ed010ca
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Dxf/wipeout : support the translation of WIPEOUT entities Jwohnlf/gdal
3 participants
@atlight @rouault @Jwohnlf