Keepout zones merge, even when of different type (exclude vias vs. exclude zones, for instance)

Bug #1789062 reported by Franck78
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
KiCad
Fix Released
High
Seth Hillbrand

Bug Description

Hello,

Create two keepout area not overlapping

Move one of them to overlap a bit with the other.

Nothing happens.
Edit one of them, they merge.

=>they merge even if they are not identical type.
Z1:via
Z2:pours

and it will result in a merged zone of an undefined type
=>a local message should warn
=>the sum of restrictions should apply (here vias+pours)

After that, rearrange some segments of the new zone does not merge them.
(but it does simplify during the join)

Automatic merging of zones should happen only if the edit follows the move with no other operations in between.
Otherwise an explicit merge must be requested.

For copper zone, more or less same problem of selection of active options

While testing bug #1513613 that does not happen for me

Application: kicad
Version: 5.0.0, release build
Libraries:
    wxWidgets 3.0.2
    libcurl/7.37.0 OpenSSL/1.0.2j zlib/1.2.8 libidn/1.28 libssh2/1.4.3
Platform: Linux 4.4.143-65-default x86_64, 64 bit, Little endian, wxGTK
Build Info:
    wxWidgets: 3.0.2 (wchar_t,STL containers,compatible with 2.8) GTK+ 2.24
    Boost: 1.61.0
    OpenCASCADE Community Edition: 6.9.1
    Curl: 7.37.0
    Compiler: GCC 4.8.5 with C++ ABI 1002

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=OFF

Tags: pcbnew
tags: added: pcbnew
Changed in kicad:
milestone: none → 5.1.0
Jeff Young (jeyjey)
summary: - joining zone
+ Keepout zones merge, even when of different type (exclude vias vs.
+ exclude zones, for instance)
Changed in kicad:
status: New → Triaged
importance: Undecided → High
Revision history for this message
KiCad Janitor (kicad-janitor) wrote :

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

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

You fixed priorities, ok.

But doing automatic merge without any kind of warning just because you edited the zone is not ok when the edit intervene say, 10 actions after the move....

-or you merge asap when overlapping detected
-or you merge on request

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.