Zone - thermal spoke width cannot be same as minimum width

Bug #1835674 reported by Jakub Kozdon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Fix Committed
Undecided
Jeff Young

Bug Description

It is possible to set thermal spoke width > minimum width

---

Application: KiCad
Version: (5.1.0-1168-g9328ec8b5), release build
Libraries:
    wxWidgets 3.0.2
    libcurl/7.52.1 OpenSSL/1.0.2r zlib/1.2.8 libidn2/0.16 libpsl/0.17.0 (+libidn2/0.16) libssh2/1.7.0 nghttp2/1.18.1 librtmp/2.3
Platform: Linux 4.9.0-8-amd64 x86_64, 64 bit, Little endian, wxGTK
Build Info:
    wxWidgets: 3.0.2 (wchar_t,wx containers,compatible with 2.8) GTK+ 2.24
    Boost: 1.62.0
    OpenCASCADE Community Edition: 6.8.0
    Curl: 7.52.1
    Compiler: GCC 6.3.0 with C++ ABI 1010

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

Tags: pcbnew
Revision history for this message
Jakub Kozdon (fldrivers) wrote :
Jeff Young (jeyjey)
Changed in kicad:
assignee: nobody → Jeff Young (jeyjey)
status: New → In Progress
milestone: none → 6.0.0-rc1
Revision history for this message
Nick Østergaard (nickoe) wrote :

The dialog requests you to do exactly that. Wjat is the bug here?

Revision history for this message
Jeff Young (jeyjey) wrote :

@Nick, the minimum width is the width of the smallest feature that should /not/ be pruned. We were treating it as the largest feature which /would/ be pruned (which should be min_width - 1).

Once you correct that, it's OK for the spoke width to be equal to the minimum width (just not smaller).

tags: added: pcbnew
Revision history for this message
KiCad Janitor (kicad-janitor) wrote :

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

Changed in kicad:
status: In Progress → Fix Committed
Revision history for this message
Jakub Kozdon (fldrivers) wrote :

Works as expected. Thank you.

Revision history for this message
Piotr Gałka (piotrgalka) wrote :

I just wonted to report (for 5.1.4) exactly the same title bug as this one, but I see that here title is not what in in the text.

The title: "can not be the same" - I wonted to report exactly this.
The bug text: "can be bigger".

5.1.4 is later than information here that it was fixed so I assume that bug described in text was fixed, but not the bug from title.

I think spoke equal to minimum should be allowed but if you try to set the same values you get message that spoke must be greater.

Application: KiCad
Version: (5.1.4)-1, release build
Libraries:
    wxWidgets 3.0.4
    libcurl/7.61.1 OpenSSL/1.1.1 (WinSSL) zlib/1.2.11 brotli/1.0.6 libidn2/2.0.5 libpsl/0.20.2 (+libidn2/2.0.5) nghttp2/1.34.0
Platform: Windows 7 (build 7601, Service Pack 1), 64-bit edition, 64 bit, Little endian, wxMSW
Build Info:
    wxWidgets: 3.0.4 (wchar_t,wx containers,compatible with 2.8)
    Boost: 1.68.0
    OpenCASCADE Community Edition: 6.9.1
    Curl: 7.61.1
    Compiler: GCC 8.2.0 with C++ ABI 1013

Build settings:
    USE_WX_GRAPHICS_CONTEXT=OFF
    USE_WX_OVERLAY=OFF
    KICAD_SCRIPTING=ON
    KICAD_SCRIPTING_MODULES=ON
    KICAD_SCRIPTING_PYTHON3=OFF
    KICAD_SCRIPTING_WXPYTHON=ON
    KICAD_SCRIPTING_WXPYTHON_PHOENIX=OFF
    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
Jeff Young (jeyjey) wrote :

@Piotr, you have to look at the Milestone rather than the dates the comments were posted.

This was fixed in 6.0. It's in a risky area so it wasn't backported to 5.1.

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.