Re: [Kicad-developers] Partial fix for Bug #1737132

2017-12-13 Thread Maciej Sumiński
Hi Jean-Pierre, On 12/12/2017 07:00 PM, jp charras wrote: > Hi Orson, > > Could you have a look into this patch. > > It fixes the Bug #1737132 (component tree widget cleared in lib editor after > any change in symbol), > but there is still a minor issue. > > The reason the component tree

Re: [Kicad-developers] [PATCH] Add double-click handling to disambiguation menu

2017-12-11 Thread Maciej Sumiński
I tested the patch on Linux and it works as advertised. Well done, we need the same thing for GAL too. Cheers, Orson On 12/06/2017 07:05 PM, Seth Hillbrand wrote: > ​The attached patch fixes https://bugs.launchpad.net/kicad/+bug/1154020 > > The current double-click handler in draw_panel​ gets

Re: [Kicad-developers] macOS & New Zone Filling?

2017-12-11 Thread Maciej Sumiński
I applied this workaround for OSX builds. It is better not to have a progress dialog rather than freeze on zone refill. It is a temporary change until we get the hardware where we can debug the problem properly. Regards, Orson On 12/09/2017 03:56 AM, Tomasz Wlostowski wrote: > On 09/12/17 01:33,

Re: [Kicad-developers] [PATCH] bugfix: OPENGL_GAL::drawPolygon did not respect fill settings

2017-12-11 Thread Maciej Sumiński
, but perhaps you may find useful either uncrustify or clang-format - we provide configuration files for both (uncrustify.cfg and _clang-format in the root directory). Cheers, Orson On 12/11/2017 10:20 AM, Andreas Buhr wrote: > On 12/11/2017 10:11 AM, Maciej Sumiński wrote: >> >> There

Re: [Kicad-developers] [PATCH] bugfix: OPENGL_GAL::drawPolygon did not respect fill settings

2017-12-11 Thread Maciej Sumiński
Andreas, There is a minor code formatting violation, that I will fix. I suppose that unfilled polygons have not been used so far, therefore the issue could not be observed. If you use them, can you confirm the behavior is coherent with the cairo backend? Cheers, Orson On 12/08/2017 12:53 PM,

Re: [Kicad-developers] [PATCH] Bugfix: EDA_RECT to BOX2I generated wrong boxes

2017-12-11 Thread Maciej Sumiński
Hi Andreas, Good catch, it must have been like this since the GAL origin. I have applied your patch to the master branch. Thank you! Cheers, Orson On 12/10/2017 11:08 PM, Andreas Buhr wrote: > Dear Kicad developers, > > The "operator BOX2I() const" of EDA_RECT produced wrong boxes, usually >

Re: [Kicad-developers] [PATCH] Use explicit shift in hotkeys instead of mixed-case characters

2017-12-11 Thread Maciej Sumiński
For the record, the patch has been already pushed. Thank you Jon! Regards, Orson On 12/09/2017 07:37 PM, Jon Evans wrote: > A minor fix due to how wxWidgets parses menu items. > > Fixes: https://bugs.launchpad.net/kicad/+bug/1737303 > > -Jon > > > >

Re: [Kicad-developers] [PATCH] Fix AlignLeft / AlignRight when using Flip Board view

2017-12-11 Thread Maciej Sumiński
Hi Jon, Thank you for the patch, I have pushed it to the master branch. Regards, Orson On 12/09/2017 08:18 PM, Jon Evans wrote: > Hi all, > > This is a "band-aid" patch for this bug: > https://bugs.launchpad.net/kicad/+bug/1734377 > > When flip board view is active, item bounding boxes are

Re: [Kicad-developers] UTF8 class and wchar_t issues fix

2017-12-08 Thread Maciej Sumiński
https://bugs.launchpad.net/kicad/+bug/1736083 as fix committed. > > On 12/08/2017 08:50 AM, Maciej Sumiński wrote: >> Hi Wayne, >> >> I have tested it and indeed it fixes the problem with Eagle import as well. >> >> Cheers, >> Orson >> >&

Re: [Kicad-developers] [PATCH] fix double free and memory leak in SHAPE_POLY_SET

2017-12-08 Thread Maciej Sumiński
Hi Andreas, Thank you for the patch. The proposed changes are reasonable, therefore I merged your patch to the master branch. Cheers, Orson On 12/08/2017 12:37 PM, Andreas Buhr wrote: > Dear Kicad developers, > > the attached patch fixes two bugs in SHAPE_POLY_SET. > > There were two problems

Re: [Kicad-developers] UTF8 class and wchar_t issues fix

2017-12-08 Thread Maciej Sumiński
Cheers, > > Wayne > > On 12/08/2017 08:38 AM, Maciej Sumiński wrote: >> Hi Jean-Pierre, >> >> Thank you for the fix, I think you are right. I applied the patch to the >> master branch. >> >> Regards, >> Orson >> >> On 12/08/2017 01

Re: [Kicad-developers] UTF8 class and wchar_t issues fix

2017-12-08 Thread Maciej Sumiński
Hi Jean-Pierre, Thank you for the fix, I think you are right. I applied the patch to the master branch. Regards, Orson On 12/08/2017 01:53 PM, jp charras wrote: > Hi Orson > > Could you have a look into this patch. > > I am thinking it is also a fix for bug 1737143. > > Currently our UTF8

Re: [Kicad-developers] Listing CMake build options (was: Simulator towards 5.0)

2017-12-07 Thread Maciej Sumiński
es defined in cmake options() or > if they get lumped in with all of the other variables. > > Cheers, > > Wayne > > On 12/7/2017 5:38 AM, Maciej Sumiński wrote: >> Oops, fixed version attached. Thank you for checking. >> >> Regards, >> Orson >> >>

Re: [Kicad-developers] Simulator towards 5.0

2017-12-07 Thread Maciej Sumiński
Oops, fixed version attached. Thank you for checking. Regards, Orson On 12/07/2017 11:35 AM, Clemens Koller wrote: > Hi! > > FYI: Your patch also touches OPENGL_GAL::BitmapText()... > > Regards, > Clemens > > On 2017-12-07 09:27, Maciej Sumiński wrote: >> Th

Re: [Kicad-developers] [PATCH] BuildConnectivity and re-DisplayBoard in python Refresh

2017-12-07 Thread Maciej Sumiński
I think this patch solves one of the problems the scripting interface users face now. Are there any objections to merging the patch? Does any Python developer care to comment whether it is a requested change? Regards, Orson On 12/03/2017 11:54 AM, miles mccoo wrote: > Refresh is used in the

Re: [Kicad-developers] [PATCH] Prevent multiple item delete when routing (fixes lp:1715158)

2017-12-07 Thread Maciej Sumiński
Hi Seth, Thank you very much for the fix. I have just pushed your patch to the master branch. Regards, Orson On 12/07/2017 05:13 AM, Seth Hillbrand wrote: > ​The attached patch corrects https://bugs.launchpad.net/kicad/+bug/1715158 > where pressing delete in pcbnew GAL canvas will delete _both_

Re: [Kicad-developers] Simulator towards 5.0

2017-12-07 Thread Maciej Sumiński
The attached patch lists options that have "KICAD" in their name when calling CMake: -- Build configuration: -- KICAD_BIN=bin -- KICAD_BRANCH_NAME= -- KICAD_DATA=share/kicad -- KICAD_DEMOS=share/kicad/demos -- KICAD_DOCS=share/doc/kicad -- KICAD_INSTALL_DEMOS=ON --

Re: [Kicad-developers] Signal integrity simulation within Kicad

2017-12-07 Thread Maciej Sumiński
Hi Andreas, I am sorry for late response, I must have lost your e-mail. Still, the answers may help you in the future. On 12/05/2017 03:42 PM, Andreas Buhr wrote: > Dear Kicad developers, > > I am browsing the Kicad source code to get an idea how to integrate a > simulation tool. > > I'd like

Re: [Kicad-developers] Zone filling

2017-12-06 Thread Maciej Sumiński
and those are only excuses. > > Regards > > Heikki > > > On Wed, Dec 6, 2017 at 12:12 PM, Maciej Sumiński <maciej.sumin...@cern.ch> > wrote: > >> Hi Heikki, >> >> Your branch contains a lot of code that is not in the official >> rep

Re: [Kicad-developers] Simulator towards 5.0

2017-12-06 Thread Maciej Sumiński
The subtle difference is your resolve wx dependency with 'apt-get install libwxgtk' and to get libngspice you need to cast a few spells when there is no package. On 12/05/2017 06:02 PM, Nick Østergaard wrote: > But it would also fail to build if you did not install wx. > > Den 5. dec. 2017 5.21

Re: [Kicad-developers] Zone filling

2017-12-06 Thread Maciej Sumiński
Hi Heikki, Your branch contains a lot of code that is not in the official repository and should not be merged (different connectivity algorithm, alternative way for via stitching interleaved with numerous #ifdef). There is no easy way to apply your changes to the master branch, as they have

Re: [Kicad-developers] Simulator towards 5.0

2017-12-05 Thread Maciej Sumiński
05:18 PM, Nick Østergaard wrote: > Isn't it good enough to mention it under the KiCad Build Configuration > Options in the devdocs as it is already? > > > 2017-12-05 16:56 GMT+01:00 Maciej Sumiński <maciej.sumin...@cern.ch>: > >> If everyone agrees that Spice

Re: [Kicad-developers] Fwd: Questions on BOARD_COMMIT

2017-12-05 Thread Maciej Sumiński
Hi Miles, On 12/04/2017 09:23 AM, miles mccoo wrote: >> Is there a document the explains Board commit? An email thread about it > >>> perhaps? >> >> There are some comments in include/commit.h. Perhaps it should be extended. >> >> > Yes, I think so. A small description of how this class is

Re: [Kicad-developers] Simulator towards 5.0

2017-12-05 Thread Maciej Sumiński
If everyone agrees that Spice simulator should be enabled by default, then please display a note saying it is optional and might be disabled for cases when it is not found by CMake. Regards, Orson On 12/05/2017 03:47 PM, Nick Østergaard wrote: > If they are not available for some reason the

Re: [Kicad-developers] Questions on BOARD_COMMIT

2017-11-30 Thread Maciej Sumiński
Hi Wayne, On 11/29/2017 06:35 PM, Wayne Stambaugh wrote: [snip]>> KIGFX::VIEW* view = s_PcbEditFrame->GetGalCanvas()->GetView(); >> item->MethodThatModifiesTheItemState(); >> view->Update( item ); > > Is this safe? What happens when pcbnew is using the legacy canvas? This is safe as long as

Re: [Kicad-developers] [PATCH] Restore "Flip Board" checkbox state when switching canvases

2017-11-29 Thread Maciej Sumiński
Thank you Jon! I pushed your patch to the master branch. Regards, Orson On 11/30/2017 02:59 AM, Jon Evans wrote: > Another quick bug fix > > -Jon > > > > ___ > Mailing list: https://launchpad.net/~kicad-developers > Post to :

Re: [Kicad-developers] Questions on BOARD_COMMIT

2017-11-29 Thread Maciej Sumiński
Hi Miles, On 11/29/2017 03:53 PM, miles mccoo wrote: > Since BOARD_COMMIT was a requested item to be added to the Python > interface, since it's related to be previous questions about getting a > redraw on the GAL canvas, and since it could make a good entry to my kicad > python tutorials, I have

Re: [Kicad-developers] [PATCH] Allow changing LAYER_PADS color and visibility (Fixes lp:1672646)

2017-11-28 Thread Maciej Sumiński
Another good patch today, thank you! I have pushed it to the master branch. Regards, Orson On 11/28/2017 02:20 AM, Jon Evans wrote: > Hi all, > > This adds LAYER_PADS to the render settings that are accessible to the > user, meaning you can change the color or hide this layer now. > > Best, >

Re: [Kicad-developers] [PATCH] Preserve color scheme when switching to legacy (Fixes lp:1670669)

2017-11-28 Thread Maciej Sumiński
Hi Jon, Thank you for the patch. It works as in the description, therefore I pushed it to the master branch. Regards, Orson On 11/28/2017 04:13 AM, Jon Evans wrote: > Hi all, > > This patch changes the color scheme behavior when switching between GAL and > legacy canvases. Previously, the

Re: [Kicad-developers] [PATCH] Reset tool transitions even when already active (Fixes lp:1733224)

2017-11-28 Thread Maciej Sumiński
Thank you Jon, I think your patch makes sense. I have just pushed it to the master branch. Regards, Orson On 11/27/2017 02:23 AM, Jon Evans wrote: > Tom, you might want to take a look at this one. > > I was looking into this bug and noticed some odd behavior and I'm not quite > sure what the

Re: [Kicad-developers] Python: time_t needs mapping; tentative patch

2017-11-28 Thread Maciej Sumiński
On 11/28/2017 09:02 AM, jp charras wrote: > Le 28/11/2017 à 00:27, Tomasz Wlostowski a écrit : >> On 27/11/17 21:18, Wayne Stambaugh wrote: >>> On 11/27/2017 2:56 PM, jp charras wrote: Le 27/11/2017 à 20:18, Tomasz Wlostowski a écrit : > On 27/11/17 17:05, Henner Zeller wrote: >> Yes,

Re: [Kicad-developers] Potrace update (AGAIN)

2017-11-27 Thread Maciej Sumiński
I wonder why do we even keep potrace in the source tree. It is available in mingw, brew and major Linux distributions, so it does not need to be build manually. Regards, Orson On 11/27/2017 03:58 AM, Simon Wells wrote: > U I *think* its jp who does the potrace stuff but there has been

Re: [Kicad-developers] [PATCH] Math expression support for pad editor.

2017-11-24 Thread Maciej Sumiński
, Orson On 11/24/2017 10:35 AM, Maciej Sumiński wrote: > Thank you Michael, I have just updated the files and pushed to my > branch. I am about to test the code on Windows and check for decimal > separator character issues. > > Cheers, > Orson > > On 11/24/2017 09:04 AM, M

Re: [Kicad-developers] New feature notifications

2017-11-24 Thread Maciej Sumiński
To have a concrete text that we can discuss, I propose an update to the Contribute->Developers page with the attached patch. What do you think? Cheers, Orson On 11/22/2017 04:21 PM, Wayne Stambaugh wrote: > On 11/22/2017 10:16 AM, Maciej Sumiński wrote: >> On 11/22/2017 03:55 PM, Way

Re: [Kicad-developers] [PATCH] Eeschema automatic manage junctions

2017-11-24 Thread Maciej Sumiński
Hi Seth, I tested the branch and I consider it a significant improvement to the way junctions are handled. I confirm it fixes the four bugs mentioned in patch 0011, apart from a single case when a parallel component is dragged out. In this case junctions are not still not auto deleted (see [1],

Re: [Kicad-developers] Getting kicad to work with wxPython Phoenix

2017-11-24 Thread Maciej Sumiński
Hi Miles, On 11/24/2017 10:19 AM, miles mccoo wrote: > in reply to Wayne's request to run the footprint wizard > > I run the footprint wizard; it seems to run fine. > when I press the 3D view button, I get a popup complaining about not > finding the wx gtk2 library > "10:14:11:

Re: [Kicad-developers] [PATCH] support "disable autopan" with gal canvas

2017-11-24 Thread Maciej Sumiński
Thank you Julius, I have just pushed your patch to the master branch. I added the 'Fixes:' line to the commit message. Regards, Orson On 11/23/2017 11:07 PM, Julius Schmidt wrote: > This patch adds support for the "disable autopan" feature to the gal > canvas. > > I'm not entirely sure this is

Re: [Kicad-developers] [PATCH] Math expression support for pad editor.

2017-11-24 Thread Maciej Sumiński
Thank you Michael, I have just updated the files and pushed to my branch. I am about to test the code on Windows and check for decimal separator character issues. Cheers, Orson On 11/24/2017 09:04 AM, Michael Geselbracht wrote: > Hi, > I have added some comments and examples to the code. The

Re: [Kicad-developers] Getting kicad to work with wxPython Phoenix

2017-11-23 Thread Maciej Sumiński
Hi Miles, On 11/23/2017 04:17 PM, miles mccoo wrote: > In a recent thread > on this list, > it was mentioned that kicad may need to drop support for SWIG/Python due to > wx and wxPython limitations. > > Perhaps I misinterpreted what

Re: [Kicad-developers] [PATCH] Math expression support for pad editor.

2017-11-23 Thread Maciej Sumiński
Just to be clear, these patches are not critical for the feature freeze. If I manage to get them merged before the freeze is announced - cool, otherwise we will have a reward for our dedicated nightly build testers in v6. Regards, Orson On 11/23/2017 04:28 PM, Maciej Sumiński wrote: >

Re: [Kicad-developers] [PATCH] Math expression support for pad editor.

2017-11-23 Thread Maciej Sumiński
ing which is probably > undesired (15.12356 -> 15.1236). > 2. The decimal separator is set to '.' the input language settings by the > user are not taken into account. > 3. I did not check for memory leaks or buffer overflows. > > - Michael > > > On Thu, Nov 23, 2017 at 2:3

Re: [Kicad-developers] [PATCH] Math expression support for pad editor.

2017-11-23 Thread Maciej Sumiński
We still do not have the feature freeze, so there is still a chance to shove it to the master branch. I feel a bit uneasy about exprtk (over 1 MB header file), but Michael's parser approach seems like a good solution to me. We can extend it later as the lemon format is not very complex. I took

Re: [Kicad-developers] [PATCH] Fix aperture macro hit testing (Fixes lp:1730249)

2017-11-23 Thread Maciej Sumiński
For the record: an updated version of this patch has been already comitted. Thank you Jon! Regards, Orson On 11/11/2017 11:22 PM, Jon Evans wrote: > Hi all, > > This patch fixes a bug reported by Nick in GerbView GAL, where aperture > macro hit testing wasn't quite right. > >

Re: [Kicad-developers] [PATCH] Allow items to be moved from all anchor points

2017-11-23 Thread Maciej Sumiński
Hi Jose, Thank you for fixing the problem. I have just pushed your patch to the master branch. Regards, Orson On 11/20/2017 05:29 PM, José Ignacio wrote: > A change in commit 57310001350 caused kicad to stop allowing users > to move items like footprints, arcs and lines using anchors other >

Re: [Kicad-developers] [PATCH] Fix for 3D model offset

2017-11-23 Thread Maciej Sumiński
I think the master branch already has enough new features to make its output incompatible with v4. 'Offset' approach seems the easiest and cleanest solution IMHO. Cheers, Orson On 11/22/2017 11:16 PM, Oliver Walters wrote: > I understand this will break "forwards compatibility" if opening a v5

Re: [Kicad-developers] New feature notifications

2017-11-22 Thread Maciej Sumiński
On 11/22/2017 04:01 PM, Marco Ciampa wrote: > On Wed, Nov 22, 2017 at 02:55:35PM +0100, Maciej Sumiński wrote: >> On 11/22/2017 02:48 PM, Marco Ciampa wrote: >> [snip] >>> Please please please, >>> >>> when some new feature is accepted (code merged in mast

Re: [Kicad-developers] New feature notifications

2017-11-22 Thread Maciej Sumiński
On 11/22/2017 03:55 PM, Wayne Stambaugh wrote: > On 11/22/2017 08:55 AM, Maciej Sumiński wrote: >> On 11/22/2017 02:48 PM, Marco Ciampa wrote: >> [snip] >>> Please please please, >>> >>> when some new feature is accepted (code merged in master branc

Re: [Kicad-developers] New feature notifications (was: Library Editor refactor)

2017-11-22 Thread Maciej Sumiński
On 11/22/2017 02:48 PM, Marco Ciampa wrote: [snip] > Please please please, > > when some new feature is accepted (code merged in master branch), either > write it in the dev (master) branch of the documentation or somewhere to > enable doc writers to describe the new feature... (wiki? txt in the

Re: [Kicad-developers] terms clarification

2017-11-22 Thread Maciej Sumiński
I wonder if there is a distinction in naming an instance and its model, like object vs class in C++. On 11/22/2017 02:31 PM, Oliver Walters wrote: > +1 for "symbol" and "footprint" > > On Thu, Nov 23, 2017 at 12:28 AM, Marco Ciampa wrote: > >> On Wed, Nov 22, 2017 at

Re: [Kicad-developers] Library Editor refactor

2017-11-22 Thread Maciej Sumiński
On 11/22/2017 02:28 PM, jp charras wrote: > Le 21/11/2017 à 23:49, Maciej Suminski a écrit : >> I have just managed to fix the last issue (disappearing buttons in 'Save >> Library' dialog) and pushed the code to the master branch. >> >> Thank you for testing and comments, I really appreciate it.

Re: [Kicad-developers] Library Editor refactor

2017-11-22 Thread Maciej Sumiński
ting library" > 2. Select a library from the file chooser > 3. On the dialog that says "Choose the library table to add the library", > press cancel or hit escape > > Cheers > > On Wed, Nov 15, 2017 at 3:37 AM, Maciej Sumiński <maciej.sumin...@cern.ch> &

Re: [Kicad-developers] Recent eeschema changes

2017-11-22 Thread Maciej Sumiński
On 11/22/2017 01:56 PM, Wayne Stambaugh wrote: [snip] > No it didn't. You were just lucky. IMO, the symbol cache library has > always been a disaster waiting to happen and it masked a lot of issues > with broken and modified symbol libraries. Your desire to have a single > symbol footprint link

Re: [Kicad-developers] Library Editor refactor

2017-11-22 Thread Maciej Sumiński
let's >> have a look at the abandoned patches on the mailing list and the bug >> tracker.. >> >> Regards, >> Orson >> >> On 11/14/2017 05:37 PM, Maciej Sumiński wrote: >>> The long awaited refactor of the Symbol Library Editor is ready for >>

Re: [Kicad-developers] Library Editor refactor

2017-11-21 Thread Maciej Sumiński
On 11/21/2017 05:06 PM, Nick Østergaard wrote: > 2017-11-21 10:22 GMT+01:00 Nick Østergaard <oe.n...@gmail.com>: > >> 2017-11-21 <20%2017%2011%2021> 9:43 GMT+01:00 Maciej Sumiński < >> maciej.sumin...@cern.ch>: >> >>> On 11/20/2017 11:16 PM, N

Re: [Kicad-developers] Library Editor refactor

2017-11-21 Thread Maciej Sumiński
On 11/20/2017 11:16 PM, Nick Østergaard wrote: > Hello Maciej, Hi Nick, > I have some comments: > > First of all, it looks quite neat. I am glad to hear that. Thank you for the test ride! > Secondly I noticed a small thing. When I open the symbol library editor and > it open in a small window

Re: [Kicad-developers] Library Editor refactor

2017-11-20 Thread Maciej Sumiński
reason of delay. If things go smooth, copy/paste will be >>>>> included in v5, otherwise - v6. >>>>> >>>>> Regards, >>>>> Orson >>>>> >>>>> On 17/11/2017 11:03 PM, Maciej Suminski wrote: >>>>>

Re: [Kicad-developers] [PATCH] fix MANDATORY_FIELDS comparisons

2017-11-20 Thread Maciej Sumiński
Thank you Julius, I have just merged your patch. Regards, Orson On 11/18/2017 01:32 PM, Julius Schmidt wrote: > Apologies for the delay, but here's the patch as an attachment. > > On Thu, 9 Nov 2017, Wayne Stambaugh wrote: > >> Julius, >> >> I cannot git am this patch.  Apparently, the extra

[Kicad-developers] Library Editor refactor

2017-11-14 Thread Maciej Sumiński
The long awaited refactor of the Symbol Library Editor is ready for testing! Changes (yes, it will be included in the commit message): - Component Tree widget as a user-friendly part selector - Multiple editor contexts, allowing the user to easily switch between modified parts -

Re: [Kicad-developers] Update of KiCad documentation?

2017-11-09 Thread Maciej Sumiński
ommit message policy is >>> set. I for one love reading changelogs when upgrading, and I think >>> the users are expecting this as well, and it probably will be useful >>> in the FOSDEM presentation :) >>> >>> >>> On 11/04/20

Re: [Kicad-developers] [PATCH 00/12] Minor cleanups and improvements

2017-11-07 Thread Maciej Sumiński
On 11/07/2017 01:24 AM, Wayne Stambaugh wrote: > On 11/3/2017 9:08 AM, Maciej Sumiński wrote: >> On 11/03/2017 01:38 PM, Carsten Schoenert wrote: >>> Hello Marvin, >>> >>> Am 02.11.2017 um 21:58 schrieb Marvin Schmidt: >>>> Just a set o

Re: [Kicad-developers] [PATCH] wxWidgets macOS 10.13 Patch

2017-11-07 Thread Maciej Sumiński
Hi Adam, Thank you for the (meta)patch. Would you give some more details about the required CPPFLAG? Perhaps we could do something on our side? I would rather avoid adding one more requirement to the way wxWidgets building process, unless there is really no other method. Regards, Orson On

Re: [Kicad-developers] [PATCH] Replace .wrl files with .step on export (fixes lp:1710796)

2017-11-06 Thread Maciej Sumiński
Hi Oliver, If I may suggest one thing, then I would turn: +if( ext == "wrl" || ext == "WRL" || ext == "Wrl" ) +return FMT_WRL; into: +if( ext.Lower() == "wrl" ) +return FMT_WRL; Apart from that, the patch seems fine just by reading the contents. No need to send another

Re: [Kicad-developers] [PATCH 00/12] Minor cleanups and improvements

2017-11-03 Thread Maciej Sumiński
On 11/03/2017 01:38 PM, Carsten Schoenert wrote: > Hello Marvin, > > Am 02.11.2017 um 21:58 schrieb Marvin Schmidt: >> Just a set of small code cleanups to fix a bunch of warnings >> and remove dead code. >> Other than that the 'build:' commits avoid/remove/fix the >> installation of certain

Re: [Kicad-developers] KiCad Library Download Page

2017-11-03 Thread Maciej Sumiński
On 11/03/2017 01:31 AM, Oliver Walters wrote: > Hi all, > > Stage 2 of ? of the library update project is underway. > > Idea: A downloads page for the libraries that presents individual libraries > for download, and users can preview the data available in the libraries. > > This is auto-built

Re: [Kicad-developers] Update of KiCad documentation?

2017-11-03 Thread Maciej Sumiński
On 11/03/2017 11:30 AM, Kristoffer Ödmark wrote: > I think starting small would be best in this case. The least needed > would be a "News" File, that is always refering to kicad stable. > > I propose the same way that patches with policy errors are rejected, > patches that introduces new

Re: [Kicad-developers] Update of KiCad documentation? (when to release kicad-5.0)

2017-11-01 Thread Maciej Sumiński
On 11/01/2017 09:40 AM, Carsten Schoenert wrote: > Hello Wayne, > > Am 31.10.2017 um 13:06 schrieb Wayne Stambaugh: >> We are very close to a feature freeze and creating a 5 stable branch. >> Once that is done, it typically takes a few months to make sure there >> are no critical or major bugs

Re: [Kicad-developers] Strange behavior of draw panel gal focus.

2017-10-31 Thread Maciej Sumiński
/md_Documentation_development_compiling.html [Building KiCad on Windows] 2. http://www.mingw.org/wiki/Workaround_for_GDB_Ctrl_C_Interrupt On 10/31/2017 01:44 AM, Bang He wrote: > how to debug on win7 > > On Mon, Oct 30, 2017 at 9:40 PM, Maciej Sumiński <maciej.sumin...@cern.ch> > wrote: > >> Hi Je

Re: [Kicad-developers] Strange behavior of draw panel gal focus.

2017-10-30 Thread Maciej Sumiński
Hi Jean-Pierre, Everything works as expected on Linux, with or without the patch. I recall having some other focus problems in the past, so there are more workarounds in common/draw_panel_gal.cpp. Perhaps there is a way to simplify the code related to focus handling or the event handlers setup

Re: [Kicad-developers] [PATCH] Add live footprint filtering in modview window

2017-10-30 Thread Maciej Sumiński
Hi Oliver, This is a really useful improvement and works as advertised, I vote for merging! Cheers, Orson On 10/29/2017 02:37 PM, Oliver Walters wrote: > In the footprint viewer window it is often hard to find the exact footprint > you want across multiple libraries > > The attached patch set

Re: [Kicad-developers] [PATCH] Gerbview: Zoom to selection tool duplication

2017-10-30 Thread Maciej Sumiński
GMT+03:00 Wayne Stambaugh <stambau...@gmail.com>: > >> Nor would I. >> >> On 10/2/2017 3:30 AM, Maciej Sumiński wrote: >>> Hi Konstantin, >>> >>> I would rather not remove the cursor tool from the legacy canvas. Once >>> you select zoo

Re: [Kicad-developers] GenCAD exporter fixes

2017-10-24 Thread Maciej Sumiński
Hi Seth, Thank you for your feedback, there are some answers below. On 10/17/2017 10:03 PM, Seth Hillbrand wrote: > Hi Orson- > > First, thank you for digging into this. Let me know if you would like a > copy of the GenCAD specification doc. Thank you for the offer, I am looking at GenCAD 1.4

Re: [Kicad-developers] [PATCH] Eagle schematic and project import.

2017-10-20 Thread Maciej Sumiński
charras wrote: > Le 18/10/2017 à 18:06, Maciej Sumiński a écrit : >> On 10/18/2017 04:58 PM, jp charras wrote: >>> Hi Orson, >>> >>> I just tested this last code >>> >>> The schematic conversion looks good to me, but Pcbnew crashes when trying >

Re: [Kicad-developers] [FEATURE] Add keepout areas in footprints

2017-10-19 Thread Maciej Sumiński
On 10/19/2017 05:20 PM, jp charras wrote: > Le 19/10/2017 à 16:59, Maciej Sumiński a écrit : >> On 10/19/2017 04:32 PM, jp charras wrote: >> [snip] >>> Hi Orson, >>> >>> Could you have a look into this patch? >>> It fixes the issue when a change

Re: [Kicad-developers] [FEATURE] Add keepout areas in footprints

2017-10-19 Thread Maciej Sumiński
On 10/19/2017 04:32 PM, jp charras wrote: [snip] > Hi Orson, > > Could you have a look into this patch? > It fixes the issue when a change is committed with a item belonging a > footprint. > > But I do not have very good knowledge of COMMIT classes. Hi Jean-Pierre, Thank you for looking into

Re: [Kicad-developers] [FEATURE] Add keepout areas in footprints

2017-10-19 Thread Maciej Sumiński
On 10/19/2017 11:31 AM, jp charras wrote: > Le 18/10/2017 à 16:57, Oliver Walters a écrit : >> JP, >> >> Fantastic, you have worked your magic! That works much better in pcbnew now, >> the undo/redo is >> working well. >> >> However, I still see that undo/redo for zone editing is broken in

Re: [Kicad-developers] [PATCH] Eagle schematic and project import.

2017-10-18 Thread Maciej Sumiński
On 10/18/2017 04:58 PM, jp charras wrote: > Hi Orson, > > I just tested this last code > > The schematic conversion looks good to me, but Pcbnew crashes when trying to > convert 2 of my test > boards (tested only on W7 32 bits) > > Attached these 2 boards. > > Regards, Jean-Pierre, Thank

Re: [Kicad-developers] PCBNew / Footprint editor - "Special strings"

2017-10-18 Thread Maciej Sumiński
Hi Thomas, On 10/18/2017 11:19 AM, Thomas Langås wrote: > Disclaimer: I come from the world of Altium, and might have a biased workflow. > > Is the following possible in any way in KiCad, or is there a different > workflow that > supports what I want? > > Usually, when I make components, I have

Re: [Kicad-developers] [PATCH] Eagle schematic and project import.

2017-10-18 Thread Maciej Sumiński
Hi Jean-Pierre, On 09/25/2017 04:10 PM, jp charras wrote: [snip] > I have 2 different issues. > > 1 - "Rounding" issues: > Attached a .sch from Eagle, and the same after conversion to eeschema file. > In the Eeschema file, many (most of?) wires are on the 1 mil grid (see for > instance lines

Re: [Kicad-developers] [RFC] Update Fields in eeschema

2017-10-02 Thread Maciej Sumiński
On 08/26/2017 08:01 PM, jp charras wrote: > Le 25/08/2017 à 15:37, Maciej Sumiński a écrit : >> The attached patches add to eeschema possibility of updating fields for >> components already placed on schematics sheet using library values. >> >> It is useful for users ma

Re: [Kicad-developers] [PATCH] Gerbview: Zoom to selection tool duplication

2017-10-02 Thread Maciej Sumiński
Hi Konstantin, I would rather not remove the cursor tool from the legacy canvas. Once you select zoom to selection, the only way to revert back to the cursor tool is to press Esc. The rest of the patch seems reasonable to me, though I want to wait for some other comments. Regards, Orson On

Re: [Kicad-developers] DXF Import: new patch (fixes 2 problems)

2017-10-02 Thread Maciej Sumiński
Hi Cirilo, Thank you for the patches. I have just pushed them to the master branch, but the stable branch might be still affected. I am not sure if it is a critical bug, and I really hope to see 5.0 rather than 4.0.8. Regards, Orson On 10/02/2017 02:20 AM, Cirilo Bernardo wrote: > The attached

Re: [Kicad-developers] [PATCH] Pcbnew: small fix for pad properties dialog

2017-10-02 Thread Maciej Sumiński
Hi Konstantin, Thank you for the patch, I have just pushed it to the master branch. Regards, Orson On 10/01/2017 03:36 PM, Константин Барановский wrote: > In "Pad properties" dialog on "Local Clearance and Settings" tab last > option has some difference from others. > Attached patch makes label

Re: [Kicad-developers] [PATCH] Dark Theme Fixups

2017-09-27 Thread Maciej Sumiński
I am sure that responding to this message awards me the Golden Shovel for digging out the oldest thread ever, but I think the patch indeed makes the interface look better. I changed it a bit to avoid code duplication and managed to test it on our supported problems without any issues. On behalf

Re: [Kicad-developers] [PATCH] Footprint viewer: zooming from toolbar and menu doesn't works in GAL mode

2017-09-26 Thread Maciej Sumiński
Hi Konstantin, Thank you for your patch, I have just pushed it to the master branch. Regards, Orson On 09/26/2017 03:15 PM, Константин Барановский wrote: > To reproduce: > 1) launch Pcbnew; > 2) switch to OpenGL; > 3) launch Footprint viewer (Library browser); > 4) select some footprint; > 5)

Re: [Kicad-developers] [PATCH] GerbView GAL support

2017-09-26 Thread Maciej Sumiński
Perfect, I have just committed your patch. Thank you! Regards, Orson On 09/25/2017 11:11 PM, Jon Evans wrote: > Thanks Orson. The attached patch changes the d-code LOD threshold to a > more reasonable value. > > -Jon > > On Mon, Sep 25, 2017 at 8:31 AM, Maciej Sumi

Re: [Kicad-developers] [PATCH] GerbView GAL support

2017-09-25 Thread Maciej Sumiński
cking. >> And the legacy canvas can be used with no issue. >> >> So, Orson, if you can commit this very good enhancement, please do it. >> AFAIK, remember also only the GAL canvas works on wxWidgets + GTK3, so the >> GAL canvas support is >> very important for the f

Re: [Kicad-developers] [PATCH] Right click option to change layer and render color

2017-09-25 Thread Maciej Sumiński
On 09/25/2017 10:28 AM, Seppe Stas wrote: > Cool, thanks! > > Any chance this (and/or the double click feature) could be released in > KiCad 4? I would love to start using it in the stable release... > > Seppe > > On Mon, Sep 25, 2017 at 10:01 AM, Maciej Sumiński

Re: [Kicad-developers] [PATCH] PcbNew Eagle Plugin: Remove layer restriction on some graphic items, fix undrawn items and place values on fabrication layers.

2017-09-25 Thread Maciej Sumiński
Hi Russell, Would you provide a board example that would be affected by the change? It would be very helpful to test the patch. I am not really sure whether EDGE_MODULEs drawn on copper layers will be exported to Gerbers and I am certain that they will not be taken into account during DRC or

Re: [Kicad-developers] [PATCH] Fix some performance issues

2017-09-25 Thread Maciej Sumiński
Hi Camille, I think the attached patches are sensible, even though I hope most compilers apply optimizations to avoid performance penalties whenever possible. If the attached patches are autogenerated, then I would kindly ask you to generate them when we reach the feature freeze stage. There are

Re: [Kicad-developers] [PATCH] Right click option to change layer and render color

2017-09-25 Thread Maciej Sumiński
do not have a ready binary to test. If time permits, I will test it a bit later. Regards, Orson On 09/23/2017 03:58 PM, Seppe Stas wrote: > Hey Maciej, what OS are you using? Also, do you have the same issue in KiCad > 4.0.7? > > On 22 Sep 2017, 17:41 +0200, Maciej Sumiński &

Re: [Kicad-developers] [PATCH] Right click option to change layer and render color

2017-09-22 Thread Maciej Sumiński
I retract it, apparently the same problem occurs without your patch applied. On 09/22/2017 05:39 PM, Maciej Sumiński wrote: > Hi Seppe, > > There is still one issue: the context menu works correctly if you right > click on a layer label, but if you do the same on a swatch box,

Re: [Kicad-developers] [PATCH] Right click option to change layer and render color

2017-09-22 Thread Maciej Sumiński
es >> between brackets and function call arguments along with some trailing >> white space. >> >> Thanks, >> >> Wayne >> >> [1]: >> http://docs.kicad-pcb.org/doxygen/md_Documentation_ >> development_coding-style-policy.html >>

Re: [Kicad-developers] [PATCH] Eagle schematic and project import.

2017-09-22 Thread Maciej Sumiński
On 09/20/2017 05:38 PM, jp charras wrote: > Le 20/09/2017 à 17:32, Maciej Sumiński a écrit : >> On 09/20/2017 05:29 PM, jp charras wrote: >> [snip] >>> After tests with 2 schematics I found a rounding issue in coordinates >>> (especially wires and pins): >>

Re: [Kicad-developers] [PATCH] GerbView GAL support

2017-09-22 Thread Maciej Sumiński
issue. Loading a few of my >>>> projects takes a really long time. Using the legacy canvas, I can load the >>>> attached layer in about 2 seconds on my relatively modest machine. When I >>>> use the OpenGL canvas, it takes almost a full minute. Same again when

Re: [Kicad-developers] [PATCH] Right click option to change layer and render color

2017-09-22 Thread Maciej Sumiński
d you clarify what part of the code does not compile? I built the KiCad > project manager and PCBNew without any issues, I didn't think other parts > would be impacted by this. > > Seppe > > 2017-09-22 11:21 GMT+02:00 Maciej Sumiński <maciej.sumin...@cern.ch>: > >&g

Re: [Kicad-developers] [PATCH] Right click option to change layer and render color

2017-09-22 Thread Maciej Sumiński
Hi Seppe, This is a very good idea. I would have merged the patch, but you need to implement GERBER_LAYER_WIDGET::OnLayerRightClick(), as in the current state the code does not compile. Regards, Orson On 09/21/2017 06:25 PM, Seppe Stas wrote: > This patch makes it possible to change the color

Re: [Kicad-developers] GitHub Plugin (my nemesis)

2017-09-22 Thread Maciej Sumiński
Hi Oliver, IMHO, the library-download tool is the best choice here. I agree the live download is not a good idea in many cases, but I imagine there are people taking advantage of that (e.g. a centralized footprint repository in a small company). I think the main problem now, is we are relying

Re: [Kicad-developers] Question : ZONE_CONTAINER

2017-09-22 Thread Maciej Sumiński
Hi Oliver, On 09/22/2017 08:28 AM, Oliver Walters wrote: > Thanks Jon, > > I was playing around with the idea of allowing keepout zones to exist on > multiple layers. I have the following functions working: > > - Select multiple layers for a keepout zone in KEEPOUT_PROPERTIES_DIALOG > -

Re: [Kicad-developers] 3: Net Connectivity

2017-09-21 Thread Maciej Sumiński
I ask for a bug report here as well. We plan to improve DRC, but it is not the right moment to do so, as we need to focus to finish the last features for v5 and switch to the feature freeze stage. Regards, Orson On 09/13/2017 06:04 AM, Strontium wrote: > His third criticism is with net

Re: [Kicad-developers] 2: Via Tool

2017-09-21 Thread Maciej Sumiński
Hi Steven, The via tool is still quite fresh, so as you have noticed - it might be lacking in certain cases. Would you please turn it into a bug report? Your message is already drowning in the mailing list traffic, but a bug report will stay in the tracker until it is resolved. Regards, Orson

<    1   2   3   4   5   6   7   8   9   >