Zone - thermal spoke width cannot be same as minimum width

Bug #1835674 reported by Jakub Kozdon on 2019-07-07
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

Jakub Kozdon (fldrivers) wrote :
Jeff Young (jeyjey) on 2019-07-12
Changed in kicad:
assignee: nobody → Jeff Young (jeyjey)
status: New → In Progress
milestone: none → 6.0.0-rc1
Nick Østergaard (nickoe) wrote :

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

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
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
Jakub Kozdon (fldrivers) wrote :

Works as expected. Thank you.

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

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  Edit
Everyone can see this information.

Other bug subscribers

Bug attachments