some comments to describe state of CKTrhs CKTrhsOld at various points
esp, note the split usage
CKTload() fills the right hand side of the matrix equation into CKTrhs
then the equation is solved and the result (no longer a right and side)
is stored in CKTrhs, and finally deployed in CKTrhsOld
ni/niiter.c, bug fix, CKTrhsOld is meant to return the solution
when solving
circuitmatrix . x = rhs
in function NIiter()
The result es expected to be delivered in CKTrhsOld.
CKTrhs is meant to store leftover garbage when returning from NIiter()
this can be seen most notably in CKTdump(), and in other iters,
for example NIacIter()
as a consequence the solution of the second to last iteration
was used, instead of the very last iteration.
This can be quite inaccurate if the second to last iteration
was marked CKTnoncon by somebody.
In my case this was the caused by the convlimit option in xpsice
which intentionally uses incorrect values and correctly markes
the results unusable with CKTnoncon.
Prevent a crash of example/FFT_test.cir.
Buffer 'buf' in function drawlingrid() has been too small.
Increase 'buf' size, and add some protection.
The example file examples/plot/test-small-plot.cir
shows the limits of plotting:
difference in two numbers at digit 14 is plotted well. at digit 15 is plotted with bugs. at digit 16 is reduced to integer.
consider compiling with
-O1 -D_FORTIFY_SOURCE=2
for gcc, and
/GS
for visual studio.
devices/bjt/bjttemp.c, bug fix, capacity temperature dependency
Fix a very old bug in the implementation of the
gummel-poon model formulas for the junction capacities.
(already in spice3f4)
For unknown reasons, the same formulas in the
diode mos1 mos2 mos6 jfet models
have been implemented correctly right from the beginning,
the typo was bjt specific.
Many Thanks for the concise and precise Bug Report
provided by Schabo Rumanus on the ngspice-users mailing list
> To: <email address hidden>
> Subject: [Ngspice-users] Temperatursimulation von Sperrschichtkapazitäten
Many Thanks to Dietmar Warning for his thorough verification
of this commit.
overwriting SPICE_DSTRING `dstr_p' may invalidate `s'
This bug was injected with the introduction of SPICE_DSTRING in :
> commit 76feebbbfad30a449a182367a9aa2d732c6cd587
> Date: Thu Feb 25 21:43:03 2010 +0000
> various bug fixes
cider relied on our broken ngspice variant of erfc() which behaved as
erfc_ngspice(x) = erfc(fabs(x))
Note:
apart from cider, erfc is used in
devices/ltra/ltramisc.c
always with positive argument
and
src/frontend/spec.c
again always with positive argument
Thus, the difference in implementation
never mathered anywhere except in cider