← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Three new source types added to DIALOG_SPICE_MODEL

 

Sylwester,

Please bear with us for a little, since the move to GitLab has been at the
foreground for the last few weeks the lead developers haven't had time to
sit down and review patches as often. Once we have completed the migration
to GitLab for the code repository, you can submit this as a merge request
there (we would prefer that to having it in Google Drive). We are in the
process of testing the process, so please be patient a little longer.

Thanks,
-Ian

On Sun, Dec 1, 2019 at 9:28 PM Sylwester Kocjan <s.kocjan@xxxxx> wrote:

> Hi,
>
> Evan's patch was merged, but mine with code change for those sources is
> not included in master currently. Can I kindly ask for update, is there
> something wrong with my code?
>
>
> https://drive.google.com/file/d/1x_YDsIw6FhgJFEsDgteEl5L5PGwSsCiz/view?usp=sharing
>
>
> Best regards,
> Sylwester
>
> On 13/11/2019 22:48, Wayne Stambaugh wrote:
> > Hi Evan,
> >
> > I think keeping the included project symbol library is the safest bet in
> > case the global library changes.  We had to continually update demos in
> > the past so this is the path of least resistance.
> >
> > Cheers,
> >
> > Wayne
> >
> > On 11/13/19 4:24 PM, Evan Shultz wrote:
> >> Hi Wayne,
> >>
> >> Thanks!
> >>
> >> Yes it is! It was merged in the GH PR I sent above, and is currently
> >> at https://github.com/KiCad/kicad-symbols and
> >> mirrored at http://kicad-pcb.org/libraries/download/ from the main
> KiCad
> >> website. So you can merge to 5.1.
> >>
> >> Would it be accepted if I updated the existing demo projects with
> >> symbols from this lib instead of the "unique" symbols used in the demo
> >> projects now?
> >>
> >> On Wed, Nov 13, 2019 at 1:18 PM Wayne Stambaugh <stambaughw@xxxxxxxxx
> >> <mailto:stambaughw@xxxxxxxxx>> wrote:
> >>
> >>      Hi Evan,
> >>
> >>      I merged your patch into the master branch.  Thank you for your
> >>      contribution to KiCad.  Just out of curiosity, is the
> Simulation_SPICE
> >>      library shipped with the 5.1 build?  If so, I can merger this demo
> into
> >>      the 5.1 branch.  Users might find it useful to demo the sources
> provided
> >>      by this library.  If not, then there is no reason to merge it into
> 5.1.
> >>
> >>      Cheers,
> >>
> >>      Wayne
> >>
> >>      On 11/12/19 2:55 PM, Evan Shultz wrote:
> >>      > Hi Wayne,
> >>      >
> >>      > Attached is the patch for the new V and I source simulation demo
> >>      project
> >>      > I mentioned. For reference, it was initially posted
> >>      > at https://github.com/KiCad/kicad-symbols/pull/1473.
> >>      >
> >>      > If you like, I can also update the existing simulation demos with
> >>      > 'standard' library components in the Simulation_SPICE directory
> >>      instead
> >>      > of the 'one-off' symbols used now.
> >>      >
> >>      > Sorry for the hijacking.
> >>      >
> >>      > Evan
> >>      >
> >>      > On Tue, Nov 5, 2019 at 5:31 AM Wayne Stambaugh
> >>      <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>
> >>      > <mailto:stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>>
> wrote:
> >>      >
> >>      >     Hi Even,
> >>      >
> >>      >     I was cleaning my inbox and I noticed I hadn't commented on
> your
> >>      >     request.  If you have a demo project that utilizes the spice
> >>      library
> >>      >     changes you made, I would be more than happy to include it in
> >>      master.
> >>      >     Please send a patch with demo project and I will merge it
> >>      after I've
> >>      >     tested it and made any comments.
> >>      >
> >>      >     Cheers,
> >>      >
> >>      >     Wayne
> >>      >
> >>      >     On 10/23/19 3:55 PM, Evan Shultz wrote:
> >>      >     > Hi,
> >>      >     >
> >>      >     > Apologies for the slight tangent...
> >>      >     >
> >>      >     > When I submitted symbols for all V and I sources supported
> >>      by ngspice
> >>      >     > at https://github.com/KiCad/kicad-symbols/pull/1473, I
> created a
> >>      >     > simulation schematic to show off their output waveforms.
> >>      Some of them
> >>      >     > can use the source control features here. I thought this
> >>      might be
> >>      >     > helpful when testing this patch
> >>      >     >
> >>      >     > It may also be useful to include the SPICE source
> simulation
> >>      as a
> >>      >     demo.
> >>      >     > If so, should I make a bug on launchpad?
> >>      >     >
> >>      >     > On Sun, Oct 20, 2019 at 2:00 PM Sylwester Kocjan
> >>      <s.kocjan@xxxxx <mailto:s.kocjan@xxxxx>
> >>      >     <mailto:s.kocjan@xxxxx <mailto:s.kocjan@xxxxx>>
> >>      >     > <mailto:s.kocjan@xxxxx <mailto:s.kocjan@xxxxx>
> >>      <mailto:s.kocjan@xxxxx <mailto:s.kocjan@xxxxx>>>> wrote:
> >>      >     >
> >>      >     >     On 20/10/2019 20:53, Seth Hillbrand wrote:
> >>      >     >     > On 2019-10-20 08:36, Sylwester Kocjan wrote:
> >>      >     >     >> Hi,
> >>      >     >     >>
> >>      >     >     >> Here is a patch with three additional source models
> >>      >     implemented:
> >>      >     >     SFFM,
> >>      >     >     >> AM and Random.
> >>      >     >     >>
> >>      >     >     >> I hope this looks ok, cause I had troubles with
> >>      configuring
> >>      >     >     >> clang-format on Windows. If no, I can correct.
> >>      >     >     >>
> >>      >     >     >> Best regards,
> >>      >     >     >> Sylwester
> >>      >     >     >>
> >>      >     >     >> _______________________________________________
> >>      >     >     >> Mailing list:
> https://launchpad.net/~kicad-developers
> >>      >     >     >> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>      <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >>      >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>      <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
> >>      >     >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>      <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >>      >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>      <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>>
> >>      >     >     >> Unsubscribe :
> https://launchpad.net/~kicad-developers
> >>      >     >     >> More help   : https://help.launchpad.net/ListHelp
> >>      >     >     >
> >>      >     >     > Hi Sylwester-
> >>      >     >     >
> >>      >     >     > There is something odd with your patch.  See the
> attached
> >>      >     >     screenshot.  I
> >>      >     >     > cannot apply this to test as there are a number of
> unknown
> >>      >     >     characters at
> >>      >     >     > the end of lines and the end of the file.
> >>      >     >     >
> >>      >     >     > -Seth
> >>      >     >     >
> >>      >     >     > Seth Hillbrand
> >>      >     >     > KiCad Services Corporation
> >>      >     >     > https://www.kipro-pcb.com
> >>      >     >     > +1 530 302 5483 | +1 212 603 9372
> >>      >     >
> >>      >     >     Hi Seth,
> >>      >     >
> >>      >     >     You are right, this patch is quite big and was
> truncated
> >>      by mail
> >>      >     >     servers, something gone wrong during sending mail. I've
> >>      put it to
> >>      >     >     google
> >>      >     >     drive:
> >>      >     >
> >>      >     >
> >>      >
> >>
> https://drive.google.com/file/d/1x_YDsIw6FhgJFEsDgteEl5L5PGwSsCiz/view?usp=sharing
> >>      >     >
> >>      >     >
> >>      >     >     Best regards,
> >>      >     >     Sylwester
> >>      >     >
> >>      >     >     _______________________________________________
> >>      >     >     Mailing list: https://launchpad.net/~kicad-developers
> >>      >     >     Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>      <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >>      >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>      <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
> >>      >     >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>      <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >>      >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>      <mailto: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
> >>      <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >>      >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>      <mailto: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
> >>      <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >>      >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> >>      <mailto: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
>

References