KiCad

footprint wizard parameter method SetValue does not update "input" field

Bug #1846404 reported by Rene Poeschl
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Fix Committed
Undecided
jean-pierre charras
KiCad 5.1.6

Bug Description

Calling set value does update the value i get inside python and also the state displayed in the output window.
But it does not update the state of the input field.

Attached test program always ensures that param and inverse are opposite (simulated radio button)

---

Application: ModEdit
Version: 5.1.4-3.fc30, release build
Libraries:
    wxWidgets 3.0.4
    libcurl/7.65.3 OpenSSL/1.1.1d-fips zlib/1.2.11 brotli/1.0.7 libidn2/2.2.0 libpsl/0.20.2 (+libidn2/2.0.5) libssh/0.9.0/openssl/zlib nghttp2/1.39.2
Platform: Linux 5.2.17-200.fc30.x86_64 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.69.0
    OpenCASCADE Community Edition: 6.9.1
    Curl: 7.65.3
    Compiler: GCC 9.1.1 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

See original description
Tags: pcbnew python
Revision history for this message
Rene Poeschl (poeschlr) wrote :
description: updated
Revision history for this message
Rene Poeschl (poeschlr) wrote :
summary: - footprint wizard parameter set value does not update "input" field
+ footprint wizard parameter SetValue does not update "input" field
summary: - footprint wizard parameter SetValue does not update "input" field
+ footprint wizard parameter method SetValue does not update "input" field
Michael Kavanagh (michaelkavanagh)
tags: added: pcbnew python
Revision history for this message
Rene Poeschl (poeschlr) wrote :

As feared "SetParameterValues" http://docs.kicad-pcb.org/doxygen-python/pcbnew_8py_source.html#l03466 does not work either.

I noticed that switching parameter page triggers the update however.

Revision history for this message
Rene Poeschl (poeschlr) wrote :

And ResetWizard does not work either. Now i am out of ideas to be honest. (I kind of fear it is impossible to transfer my footprint generators over to wizards until either this is fixed or there is a drop down list parameter type as i request in https://bugs.launchpad.net/kicad/+bug/1846388)

Revision history for this message
Rene Poeschl (poeschlr) wrote :

I forgot to mention that i also tried pcbnew.Refresh() and pcbnew.UpdateUserInterface() both did not help.

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

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

Changed in kicad:
status: New → Fix Committed
assignee: nobody → jean-pierre charras (jp-charras)
Revision history for this message
jean-pierre charras (jp-charras) wrote :

@Rene,

Could you test my fix (for now only in master branch) and see if it is working with some other wizards.

Thanks.

Revision history for this message
Rene Poeschl (poeschlr) wrote :

Sadly fedora does not offer a nightly build version that can be installed next to the stable version. I need to run stable as i need stable to test contributions to the library. So i am sadly unable to test anything on nightly builds (without considerable extra effort like using a virtual machine).

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

@Rene,

I forgot to say you I committed the fix in 5.1.6 stable branch (commit 2b4cfd9a608).

Michael Kavanagh (michaelkavanagh)
Changed in kicad:
milestone: none → 5.1.6
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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