Oval pad occupies the whole bounding box

Bug #1843101 reported by eelik
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Fix Released
Undecided
Jeff Young

Bug Description

An oval pads seems to occupy the whole bounding box so that for example clarification menu pops up even when the cursor isn't on the pad. This alone wouldn't be very serious, but I can't put a via there for a different net because it automatically takes the net of the pad.

See the example board, and I'll attach a video, too.

Application: Pcbnew
Version: (5.1.4-62-g8d04d3903)-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.21.0 (+libidn2/2.1.1) nghttp2/1.34.0
Platform: Windows 8 (build 9200), 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

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

Turns out to be a bug in TestSegmentHit(), so it also affects tracks.

Changed in kicad:
assignee: nobody → Jeff Young (jeyjey)
status: New → In Progress
milestone: none → 6.0.0-rc1
status: In Progress → Fix Committed
tags: added: pcbnew
Revision history for this message
eelik (eelik) wrote :

Is this going to be fixed in 5.1, too?

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

Probably a good idea....

Changed in kicad:
milestone: 6.0.0-rc1 → 5.1.5
status: Fix Committed → In Progress
Revision history for this message
Seth Hillbrand (sethh) wrote :

Did this fix affect lp:1843160?

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

Ha! Not intentionally, but it does appear to have fixed it.

(While I don't pretend to understand the math the old algorithm was doing, it was doing it off of a single end-point of the segment.)

Jeff Young (jeyjey)
Changed in kicad:
status: In Progress → 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

Remote bug watches

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