pcbnew, import dxf, not handeld dxf items

Bug #1790821 reported by Nicholas Savenlid
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Fix Released
Medium
Seth Hillbrand

Bug Description

Hi

i get "DL_Solid not managed"

I am not sure why

I have the feeling this worked before.

its a simple dxf attached.

If its my fault, some hints towards to root cause is appreciated.

thanks

Application: kicad
Version: (6.0.0-rc1-dev-454-g181ce46b9), release build
Libraries:
    wxWidgets 3.0.3
    libcurl/7.54.1 OpenSSL/1.0.2l zlib/1.2.11 libssh2/1.8.0 nghttp2/1.23.1 librtmp/2.3
Platform: Windows 7 (build 7601, Service Pack 1), 64-bit edition, 64 bit, Little endian, wxMSW
Build Info:
    wxWidgets: 3.0.3 (wchar_t,wx containers,compatible with 2.8)
    Boost: 1.60.0
    OpenCASCADE Community Edition: 6.8.0
    Curl: 7.54.1
    Compiler: GCC 7.1.0 with C++ ABI 1011

Build settings:
    USE_WX_GRAPHICS_CONTEXT=OFF
    USE_WX_OVERLAY=OFF
    KICAD_SCRIPTING=ON
    KICAD_SCRIPTING_MODULES=ON
    KICAD_SCRIPTING_WXPYTHON=ON
    KICAD_SCRIPTING_ACTION_MENU=ON
    BUILD_GITHUB_PLUGIN=ON
    KICAD_USE_OCE=ON
    KICAD_USE_OCC=OFF
    KICAD_SPICE=ON

Tags: dxf import pcbnew
Revision history for this message
Nicholas Savenlid (nicholas-z) wrote :
Revision history for this message
Jeff Young (jeyjey) wrote :

Marking as High at least until we find out if it's a regression or not.

Changed in kicad:
importance: Undecided → High
Revision history for this message
Seth Hillbrand (sethh) wrote :

We've never handled solid objects. Previous DXF import silently skipped them.

Changed in kicad:
status: New → Triaged
importance: High → Wishlist
Revision history for this message
Nicholas Savenlid (nicholas-z) wrote :

But then it's a difference.
Now it aborts.

Revision history for this message
jean-pierre charras (jp-charras) wrote :

Pcbnew is not a MCAD tool, so many dxf entities cannot be managed, and are just ignored.
Now a warning message (as requested by many users) is displayed for ignored dxf entities.

However I do not have an issue to import your dxf sample: just a warning lists the ignored item (as expected), but no abort and no crash.

Revision history for this message
Nicholas Savenlid (nicholas-z) wrote :

Ok.
Maybe I misinterpreted that part.
My memory was that it was also not imported.
However this was some time
ago so can't remember the details.

Revision history for this message
m (easyw) wrote :
tags: added: dxf import pcbnew
Revision history for this message
Ian McInerney (imcinerney) wrote :

What is this report actually requesting? It isn't clear from the previous discussion.

Revision history for this message
Seth Hillbrand (sethh) wrote :

@Maurice, the file imports correctly for me.

Please add the version information for which it is broken.

Application: Pcbnew
Version: (5.1.4-113-gf2e75c96b), debug build
Libraries:
    wxWidgets 3.0.4
    libcurl/7.64.0 OpenSSL/1.1.1d zlib/1.2.11 libidn2/2.0.5 libpsl/0.20.2 (+libidn2/2.0.5) libssh2/1.8.0 nghttp2/1.36.0 librtmp/2.3
Platform: Linux 4.19.0-6-amd64 x86_64, 64 bit, Little endian, wxGTK
Build Info:
    wxWidgets: 3.0.4 (wchar_t,wx containers,compatible with 2.8) GTK+ 3.24
    Boost: 1.67.0
    OpenCASCADE Community Edition: 6.9.1
    Curl: 7.64.0
    Compiler: GCC 8.3.0 with C++ ABI 1013

Build settings:
    USE_WX_GRAPHICS_CONTEXT=OFF
    USE_WX_OVERLAY=ON
    KICAD_SCRIPTING=ON
    KICAD_SCRIPTING_MODULES=ON
    KICAD_SCRIPTING_PYTHON3=ON
    KICAD_SCRIPTING_WXPYTHON=ON
    KICAD_SCRIPTING_WXPYTHON_PHOENIX=ON
    KICAD_SCRIPTING_ACTION_MENU=ON
    BUILD_GITHUB_PLUGIN=ON
    KICAD_USE_OCE=ON
    KICAD_USE_OCC=OFF
    KICAD_SPICE=ON

Revision history for this message
Seth Hillbrand (sethh) wrote :

Oops, sorry. Disregard that. I see the issue.

We are not handling the engraving layer correctly. Remove that (and its items) and the problem goes away. It doesn't relate to the original bug report but we should still fix it.

Seth Hillbrand (sethh)
Changed in kicad:
importance: Wishlist → Medium
milestone: none → 5.1.5
Revision history for this message
KiCad Janitor (kicad-janitor) wrote :

Fixed in revision df3fabfa21c1488c2ce9c2505de9663489d677df
https://git.launchpad.net/kicad/patch/?id=df3fabfa21c1488c2ce9c2505de9663489d677df

Changed in kicad:
status: Triaged → Fix Committed
assignee: nobody → Seth Hillbrand (sethh)
Revision history for this message
Seth Hillbrand (sethh) wrote :

Bumping this to deal with the import failure in 5.1.5.

Changed in kicad:
assignee: Seth Hillbrand (sethh) → nobody
status: Fix Committed → Triaged
assignee: nobody → Seth Hillbrand (sethh)
Seth Hillbrand (sethh)
Changed in kicad:
status: Triaged → Fix Committed
Changed in kicad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.