← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Eeschema automatic manage junctions

 

Hi Orson-

Thanks for the test and the clarification of the documentation.

On the schematic, I can't seem to recreate the issue you document in the
image.  Is the example schematic one you can share?

Best-
Seth

On Fri, Nov 24, 2017 at 6:50 AM, Maciej Sumiński <maciej.suminski@xxxxxxx>
wrote:

> 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], the rightmost picture "Junction is not auto deleted"), but at least
> the overlapping wires are merged. It is not a big deal IMHO, especially
> the patch set fixes a lot of other issues.
>
> It has not been stated explicitly, but I thought we will use
> NEW/REMOVE/CHANGE tags for listing changes that are perceived by the
> user. Therefore:
>
>   CHANGE: DeleteItem removes junctions that are no longer needed.
>
> informs the user about an improvement, but:
>
>   CHANGE: DeleteItemsInList shares the code for DeleteItem.
>
> seems to be too detailed and does not need to be reflected in the user
> documentation.
>
> What do you think? I will try to come up with a short paragraph that can
> be added to the website to make things clear.
>
> Regards,
> Orson
>
> 1.
> https://launchpadlibrarian.net/213748651/KICAD%20-%20BUG%
> 20REPORT%20-%20EESCHEMA%20-%20SOME%20DRAG%20ISSUES.jpg
>
> On 11/22/2017 09:07 PM, Seth Hillbrand wrote:
> > Updated patchset for this proposal, rebased to master.  I've also updated
> > the commit messages to match the CHANGE:/NEW: format and added one new
> bug
> > from launchpad that this addresses.
> >
> > -Seth
> >
> > On Wed, Nov 8, 2017 at 3:59 AM, Nick Østergaard <oe.nick@xxxxxxxxx>
> wrote:
> >
> >> For that specific issue with the junction drawing, there is a patch in
> the
> >> thread "[Kicad-developers] [PATCH] Draw junctions last"
> >>
> >> 2017-11-03 13:12 GMT+01:00 Jon Evans <jon@xxxxxxxxxxxxx>:
> >>
> >>> I looked at fixing this and some other related things, and decided to
> >>> just wait for the GAL port. There will need to be huge refactoring of
> the
> >>> eeschema draw code as part of that effort, so putting much effort into
> >>> making the wxDC drawing better seems not worth it.
> >>>
> >>> -Jon
> >>>
> >>> On Nov 3, 2017 00:08, "Kevin Cozens" <kevin@xxxxxxxxx> wrote:
> >>>
> >>> On 2017-11-02 06:31 PM, Seth Hillbrand wrote:
> >>>
> >>>> Please let me know if there are any additional issues or suggestions
> for
> >>>> improvement.
> >>>>
> >>>
> >>> How difficult would it be to have junctions draw last on schematics?
> >>> There is a minor negative visual effect when you have a component with
> one
> >>> end joined to a wire by a junction and you replace the component.
> >>>
> >>> When you replace the component the pin of the component is now seen
> >>> extending through the round disc of the junction to the center of the
> >>> junction. I prefer to always see just the full round disc of a junction
> >>> mark even if I have replaced a component since placing the junction.
> >>>
> >>> --
> >>> Cheers!
> >>>
> >>> Kevin.
> >>>
> >>> http://www.ve3syb.ca/           |"Nerds make the shiny things that
> >>> distract
> >>> Owner of Elecraft K2 #2172      | the mouth-breathers, and that's why
> >>> we're
> >>>                                 | powerful!"
> >>> #include <disclaimer/favourite> |             --Chris Hardwick
> >>>
> >>>
> >>> _______________________________________________
> >>> Mailing list: https://launchpad.net/~kicad-developers
> >>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>> Unsubscribe : https://launchpad.net/~kicad-developers
> >>> More help   : https://help.launchpad.net/ListHelp
> >>>
> >>>
> >>>
> >>> _______________________________________________
> >>> Mailing list: https://launchpad.net/~kicad-developers
> >>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>> Unsubscribe : https://launchpad.net/~kicad-developers
> >>> More help   : https://help.launchpad.net/ListHelp
> >>>
> >>>
> >>
> >> _______________________________________________
> >> Mailing list: https://launchpad.net/~kicad-developers
> >> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> >> Unsubscribe : https://launchpad.net/~kicad-developers
> >> More help   : https://help.launchpad.net/ListHelp
> >>
> >>
> >
> >
> >
> > _______________________________________________
> > Mailing list: https://launchpad.net/~kicad-developers
> > Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> > Unsubscribe : https://launchpad.net/~kicad-developers
> > More help   : https://help.launchpad.net/ListHelp
> >
>
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>
>

Follow ups

References