Skip to content

Commit

Permalink
Tracked down an obscure error that was causing a property
Browse files Browse the repository at this point in the history
mismatch in the SkyWater sky130_fd_io__top_pwrdetv2 circuit
because a resistor with ends shorted together was being assigned
an incorrect length and width.  This was due to the similarity
in characteristics of the boundary vector between a shorted
resistor and an annular resistor.  The terminals need to be
checked for shorted ends to disambiguate the two cases.
  • Loading branch information
RTimothyEdwards committed Nov 15, 2024
1 parent 22c2222 commit 1fdca3a
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
2 changes: 1 addition & 1 deletion VERSION
Original file line number Diff line number Diff line change
@@ -1 +1 @@
8.3.499
8.3.500
11 changes: 10 additions & 1 deletion extract/ExtBasic.c
Original file line number Diff line number Diff line change
Expand Up @@ -2549,9 +2549,18 @@ extOutputDevices(def, transList, outFile)
/* Mark annular resistors as requiring extra processing */
if (extTransRec.tr_termvector[n].p_x == 0 &&
extTransRec.tr_termvector[n].p_y == 0)
isAnnular = TRUE;
{
/* Update: 11/14/2024: A snake resistor with terminals
* tied together also has a zero termvector and should
* not be treated as annular!
*/
if ((n == 0) || (extTransRec.tr_termnode[n] !=
extTransRec.tr_termnode[n - 1]))
isAnnular = TRUE;
}
}


if (n == 0)
width = length = 0;
else if (ntiles > 1)
Expand Down

0 comments on commit 1fdca3a

Please sign in to comment.