← Back to team overview

kicad-developers team mailing list archive

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

 

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




Follow ups

References