kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #42536
Re: [PATCH] simulation: allow entering signals (spice vectors) by name
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Mon, 11 Nov 2019 09:47:06 -0500
-
Autocrypt:
addr=stambaughw@xxxxxxxxx; prefer-encrypt=mutual; keydata= mQGiBEM0hxQRBAC2fNh3YOVLu1d5GZ0SbrTNldGiGnCJPLqzEnqFX9v6jmf33TMt6EmSLkl6 Wtfkoj0nVwKxcYmJkA8DX0QAokBkwNIzhSsBzQvthBLIk/5LnPVVKrEXOcL4mUyH1doKlkaE slgJozNa6Av+oavcvD02o1zJOloBbaHlNlyRt7fKswCgtIFlVjWggVH/15KfWk+Qo5JVPbME AIUBAQyL2OAx0n60AWec2WHnO9buHuG0ibtICgUMkE+2MRmYyKwYRdyVwGoIUemFuOyHp0AJ InX4T+vy2E7vkwODqjtMLfIoRkokW74Fi4nrvjlhOAw/vdq/twLbAmR9MOfPTpR4y7kQy1O2 /n+RkkRvh26vTzfbQmrH7cBJhk6aA/9Uwvu3E4zNJgHVZeS0HyWtmR1eOPPRbnkPgJTToX5O KMKzTJI/FX6kT7cFoCamitHrW3BJP4Dx+cMMsa47EGxqVTdbVJ4LjogsXTXxb+0Fn1u4zBdx x3Cer6O7+hqWy7zvpzeC6nSREjqDKa5CgHtv/GLm5uFPOmsjAsnHj2tlBrQmV2F5bmUgU3Rh bWJhdWdoIDxzdGFtYmF1Z2h3QGdtYWlsLmNvbT6IeAQTEQIAOBYhBOffs6CbblRzBkv33BtR cWlZ+CReBQJbFBS2AhsDBQsJCAcCBhUKCQgLAgQWAgMBAh4BAheAAAoJEBtRcWlZ+CReMI8A nRbrLkzp7+c2f0vX7sfg4ICX8LAKAJ9uClo4uJajmZa5zZrL2nKdZlUwIrkCDQRDNIcxEAgA gCru+3/aOC6RCjpvYC72wY+d5SmHphC6yeiV2/mOumyt5MLo/Ps2GznZr11JspqFk5K/Zpvp MMLqqjDZ39+50a2iKRQFJ6NlK+hJWMmj6eJygQrCwYo3Gjc6CqfrqUv+8VSnf/i5sIZmtOVA 4ZjML18MuBvMSsNdVLFJd5HNnYb1iOECpvqdPVh/21LLCEw7MUUGGnHBhCrmk2aJe5hFmcSN g4ldBcXrgMQBwf7aMVoobXBMFDb/IENByXn0llB7Gr2IFMRmNS9/p8s/II1Yl2bTqyX4FSz8 cfn7C9KEz7faZ7wzAcpwHFC/zs3JoAjJ0IEKdNUpIwAlKMzT3CzctwADBQf/cxpG28MKyrqk nNmq/8LQLy+x6FSYXBLjxQz9BiBNYeesDZQ6J5UbL1mjpJzMa5tLZypPYo4bbGyR22hrbyDF K7m6AcVaMIJKl98g4ukMutFfAJyRDaREH5Zl/X1P4u1Z/yaAIy9mKaNbaK1/5djNJ5wCTFen TUgAp9xdc30kGkFDdLJFp5uxDY4P0vaZiZdjUCvDM3Zjv5IzpNOfxVqTUBQNUP/BnnKhkk0p DTD6s3X8S+D0rOtEBQ8K0cwERI/E8EFa8nj0TNw4e2MYGR8wg+SxqJ7z5f0zPY0bO6G9DDFB wYCqzzPWGqdAh9vA5971TAbPERtdFybhkurozp2SfYhJBBgRAgAJBQJDNIcxAhsMAAoJEBtR cWlZ+CResHUAniULLCWiT26ieRTl7N2vS6vBo/DuAJ4m7Ss/gyiW6ybTn1ctDXAUgm2QVQ==
-
Cc:
KiCad Developers <kicad-developers@xxxxxxxxxxxxxxxxxxx>
-
In-reply-to:
<CALfiFhBToCvgommynjRRset8Dc5V1UKZi9G84sFpdGj_vnWbsQ@mail.gmail.com>
-
Openpgp:
preference=signencrypt
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0
Hi Jonaton,
I pushed your patch into the master branch. I haven't made up my mind
on the 5.1 branch. Technically this is a new feature. We have an
unwritten policy to not backport new features from the master branch.
Given that this is a fairly trivial change, I may consider applying your
patch to the 5.1 branch after 5.1.5 is released.
All that being said, I hope that this patch is a temporary fix and that
we use the information Holger provided on how to query ngspice to fetch
a list of vectors that can be plotted rather than manually entering
them. I'm sure someone is going to request this and even if they don't
I think this would provide a far better user experience.
Thank you for your contribution to KiCad.
Cheers,
Wayne
On 11/7/19 3:52 AM, Jonatan Liljedahl wrote:
> Hi,
>
> Here's an updated patch with fixed commit message. Also I removed some
> unrelated UI tweaks in sim_plot_frame that accidentally was part of
> the previous patch. This patch applies to both 5.1 and master.
>
> Cheers
>
> On Thu, Nov 7, 2019 at 12:04 AM Kymatica <j.r.liljedahl@xxxxxxxxx> wrote:
>>
>> Oh, this was for 5.1.
>> I’ll look into rebasing on master tomorrow! Sleepy time here in Sweden...
>>
>> /Jonatan
>>
>>> 6 nov. 2019 kl. 23:57 skrev Wayne Stambaugh <stambaughw@xxxxxxxxx>:
>>>
>>> This patch does not apply cleanly against master using `git am`. Did
>>> you forget to rebase against the master branch before creating your patch?
>>>
>>> Wayne
>>>
>>>> On 11/6/19 5:48 PM, Jonatan Liljedahl wrote:
>>>> Ok, now I think I've followed the policies correctly, here's the updated patch.
>>>>
>>>> Cheers
>>>>
>>>>> On Wed, Nov 6, 2019 at 10:33 PM Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
>>>>>
>>>>> Hi Jonathon,
>>>>>
>>>>> I tested your original patch on the 5.1 branch and it seems to work as
>>>>> advertised. It would be nice if there was a way to query ngspice for a
>>>>> list of vectors rather than using just the net list to populate add
>>>>> signals dialog. Maybe Holger can weigh in on this.
>>>>>
>>>>> Before you submit your updated patch, please fix your code formatting to
>>>>> follow the KiCad Coding Policy[1] and your commit message to follow the
>>>>> KiCad Commit Message[2] policy. There is a git commit hook[3] to help
>>>>> you with coding policy issues. I saw some K&R curly brace placement in
>>>>> your patch.
>>>>>
>>>>> Thanks,
>>>>>
>>>>> Wayne
>>>>>
>>>>> [1]:
>>>>> http://docs.kicad-pcb.org/doxygen/md_Documentation_development_coding-style-policy.html
>>>>> [2]: http://docs.kicad-pcb.org/doxygen/commit_messages.html
>>>>> [3]:
>>>>> http://docs.kicad-pcb.org/doxygen/md_Documentation_development_coding-style-policy.html#tools
>>>>>
>>>>> On 11/6/19 4:06 PM, Jonatan Liljedahl wrote:
>>>>>> This patch does not implement all of pt 4 in that bug report, it only
>>>>>> allows adding plots of vectors by name, including vectors created by
>>>>>> LET statements in a spice text block in the schematic (which are not
>>>>>> listed in the signals list, since that list is based on the net names,
>>>>>> not the existing spice vectors).
>>>>>>
>>>>>> However, I just realized that there's a problem with my patch, it only
>>>>>> works for V, not I. I'll take a look and update my patch.
>>>>>>
>>>>>> On Wed, Nov 6, 2019 at 9:08 PM Sylwester Kocjan <s.kocjan@xxxxx> wrote:
>>>>>>>
>>>>>>> Hi Jonatan,
>>>>>>>
>>>>>>> On 06/11/2019 13:29, Jonatan Liljedahl wrote:
>>>>>>>> useful for plotting calculated vectors ("let AB=V(a)*V(b)"),
>>>>>>>> see for example
>>>>>>>> https://forum.kicad.info/t/ac-analysis-of-op-amp-loop-gain-and-phase/19661/6
>>>>>>>>
>>>>>>>
>>>>>>> Just small remark to limit entropy in bug reports: it looks like you
>>>>>>> have implemented pt. 4 from:
>>>>>>>
>>>>>>> https://bugs.launchpad.net/kicad/+bug/1814188
>>>>>>>
>>>>>>> Maybe it's worth to update that bug report if it will be merged.
>>>>>>>
>>>>>>> Best regards,
>>>>>>> Sylwester
>>>>>>>
>>>>>>> _______________________________________________
>>>>>>> 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