luke/magic
1
0
Fork 0
mirror of https://github.com/RTimothyEdwards/magic.git synced 2026-09-07 19:36:23 +02:00
Code Issues Packages Projects Releases Wiki Activity

Commit Graph

  • e45399d347 Yet another optimization on "extresist", to remove the calls to DBPaint() and DBErase(), which are meant as high-level calls from the "paint" and "erase" commands and are not appropriate for low- level tasks like erasing all contacts out of a database. This cut maybe 40 to 50% time off of the resistance extraction. 8.3.648 R. Timothy Edwards 2026-05-22 12:02:03 -04:00
  • 7a3717b02a Another performance optimization to "extresist", this one to avoid re-parsing a linked list from the current breakpoint to the end to find all entries pointing to the same node. Instead, the change to be made is saved in a hash table and applied at the following link, so only one pass through the linked list is required. 8.3.647 R. Timothy Edwards 2026-05-21 19:55:50 -04:00
  • a062fdcfe0 Continued optimizing poorly written routines in "extresist". Changed the "ResDissolveContacts()" routine to avoid running both redundant and useless code. Changed the breakpoint sorting routine from a bubble sort to a merge sort for linked lists longer than 16, as merge sort is more efficient for long lists. 8.3.646 R. Timothy Edwards 2026-05-20 19:02:22 -04:00
  • 29447a35cd Delete duplicate magicps.pro file 8.3.645 Ahmed Nematallah 2026-05-17 03:46:10 +02:00
  • 1dcd09e20c Update the postscript file to be able to support much more stipples Ahmed Nematallah 2026-05-17 03:41:07 +02:00
  • 9626b5e8c9 Changed two sprintf() to snprint() on recommendation from an AI. R. Timothy Edwards 2026-05-19 12:21:54 -04:00
  • 256955f48e Created alternative algorithms for routines ResParallelCheck() and ResTriangleCheck(). By using hash tables, I reduced ResParallelCheck() from O(N^2) to O(N), and I reduced ResTriangleCheck() from O(N^3) to O(N^2); however, it is a bit better than that because previously there was an N^2 loop over items in the same list, so if one list happened to be very long, then N^2 would be huge; whereas now the N^2 is the length of two lists multiplied together, where both lists would have to be very long to have the same performance impact. It appears that long resistor lists are pathological and rare. For an example pathological case, the extresist runtime has been reduced from hours to seconds. 8.3.644 R. Timothy Edwards 2026-05-15 21:38:32 -04:00
  • b983e33be7 Corrected an error with the recently-added "exception" DRC rule. The logical expression for determining an excepted area was wrong for one direction, resulting in false positive DRC violations. R. Timothy Edwards 2026-05-15 11:04:23 -04:00
  • db224105a7 Found another couple of places where the environment variable HOME was checked but no check was made on whether or not the result was NULL. This resolves issue #490 in the github issue tracker. 8.3.643 R. Timothy Edwards 2026-05-13 09:46:21 -04:00
  • e5a6cf0df9 Handled the case in which environment variable HOME is not set. See github issue tracker issue #490. There were two cases: When running without the Tk console, a NULL value was being passed to a string comparison. When running with the Tk console, two variables were not initialized. Both prolems have been fixed. R. Timothy Edwards 2026-05-13 08:51:16 -04:00
  • 22e182f908 One additional modification to check for a space in the option with no following text. R. Timothy Edwards 2026-05-13 08:38:11 -04:00
  • 0013dda92d Response to issue #513 in the github issue tracker: The command- line parser does not recognize arguments that are both in the same word and also space separated; this is easy to do from, e.g., a python interpreter by setting an option as a single string as in "-d null"; it is harder to do from the command line, but can be done with quotes, as in 'magic "-d null"'. R. Timothy Edwards 2026-05-13 08:30:32 -04:00
  • 85561d0503 Corrected an error with the new "tagged" operator when used in "cifinput"; instead of cifSrTiles(), which ignores the "area" argument and searches over the entire plane, it should have been using cifSrTiles2(), which translates the area to CIF coordinates and searches only over that area. Otherwise, everything on the same layer in a cell gets tagged, rather than just connected material. 8.3.642 R. Timothy Edwards 2026-05-12 12:06:16 -04:00
  • eb83075e16 Modified EXT_DOALL to include EXT_DOUNIQUE. Prior to a recent change, unique node names could only be enforced by running the command "extract unique", and there was no way to ensure that the user would run the command as part of the extraction recipe. Now that it has been changed to an extraction option instead of a separate command, and does not permanently alter labels in the layout like the standalone command version did, it is possible to include the setting in "extract do all", making it run by default, and requiring the user to manually turn it off for certain use cases (which are relatively rare). R. Timothy Edwards 2026-05-12 10:36:36 -04:00
  • 8fea20425e Merged PR #512 from Intubun, reworking the wasm workflow. Updated the version to force a test of the workflow. R. Timothy Edwards 2026-05-12 08:57:42 -04:00
  • ad14c26597 ci(wasm): also publish on tag push Intubun 2026-05-12 11:19:11 +02:00
  • 19a2d5c57c ci(wasm): auto-publish @<owner>/magic-vlsi-wasm to GitHub Packages when VERSION changes Intubun 2026-05-12 11:13:25 +02:00
  • fb01f77755 Updated the version to go along with the merge of pull request the repo parsing on opencircuitdesign.com, since yesterday I found out that the tags were not updating, caused by a documentation change in a copy of the repository that created a merge conflict when seen by the script. Will check tomorrow that everything ran smoothly overnight. 8.3.641 R. Timothy Edwards 2026-05-11 14:23:04 -04:00
  • 9d74f4f9a7 update main.yml Enno Schnackenberg 2026-05-11 06:59:25 +02:00
  • 9af6b329de update main.yml Enno Schnackenberg 2026-05-11 06:57:32 +02:00
  • 6d3494a8fe Update extract.tcl Enno Schnackenberg 2026-05-07 07:17:45 +02:00
  • 068b93608a Update extract.tcl Enno Schnackenberg 2026-05-07 06:57:26 +02:00
  • e62626fc06 Update extract.tcl Enno Schnackenberg 2026-05-07 06:53:37 +02:00
  • 145fba9ac0 Update extract.tcl Enno Schnackenberg 2026-05-07 06:49:58 +02:00
  • 19113bec89 Update extract.tcl Enno Schnackenberg 2026-05-07 06:48:02 +02:00
  • 480c40f516 Update extract.tcl Enno Schnackenberg 2026-05-07 06:44:13 +02:00
  • 1fffc2b8b4 Update main-wasm.yml Enno Schnackenberg 2026-05-06 20:06:47 +02:00
  • 392d9c4408 Rework WASM CI structure Enno Schnackenberg 2026-05-06 19:47:08 +02:00
  • c8b50a702d Change npm examples to scmos examples Enno Schnackenberg 2026-05-06 19:16:55 +02:00
  • 354416f717 ci: add ARM WASM build job with Emscripten diagnostics to CI-aarch64 Intubun 2026-05-06 11:02:29 +02:00
  • f802d729af Annotate ClientData callbacks: /*ARGSUSED*/ + /* UNUSED */ markers Intubun 2026-05-06 10:43:34 +02:00
  • 1d402d2c5f npm: stamp version as base-YYYYMMDD.hash on every CI build Intubun 2026-05-06 10:38:06 +02:00
  • 2d6a3a8ed6 CI: publish to GitHub Packages, emsdk latest by default, add ARM WASM job Intubun 2026-05-06 07:27:29 +02:00
  • 8560027569 Add npm package and CI workflows Intubun 2026-05-04 13:32:51 +02:00
  • 02087ea3f8 cmwind: ARGSUSED/UNUSED annotation on CMWreposition (newScreenArea unused) Intubun 2026-05-06 07:01:54 +02:00
  • 8e8fada32f Add WASM entry point and Emscripten build wiring Intubun 2026-05-04 13:31:41 +02:00
  • f4c22438c6 Wire up headless / WASM mode at module boundaries Intubun 2026-05-04 13:31:11 +02:00
  • fdfb54badb graphics/grNull.c: typed no-op stubs for WASM call_indirect Intubun 2026-05-04 13:30:21 +02:00
  • 6fc7b30386 Save and restore GrDisplayStatus across long operations Intubun 2026-05-04 13:29:49 +02:00
  • 476bb5474c Add ClientData parameter to indirect-call callbacks for WASM Intubun 2026-05-04 13:29:11 +02:00
  • 537d370536 Fix five latent bugs surfaced while porting to WASM Intubun 2026-05-04 13:28:42 +02:00
  • bdc0325901 Extended the use of "ext2spice resistor tee on" to include DEV_RSUBCKT as well as (previously implemented) DEV_RES. The handling of DEV_RSUBCKT includes checking if the subcircuit has a substrate pin, and zeroing the capacitance to substrate for the resistor node, under the assumption that the parasitics are part of the resistor subcircuit model. 8.3.640 R. Timothy Edwards 2026-05-09 20:05:52 -04:00
  • fa80458d80 Ensure layer name compatibility in wrapper.tcl Ahmed Nematallah 2026-05-02 22:50:25 +02:00
  • c3ee33e9a1 tcl/tk: Fix layer toolbar icon synchronization on left-click Ahmed Nematallah 2026-05-02 18:09:18 +02:00
  • 900f6d7b0f Ensure all cells are loaded into memory in netlist_to_spice to avoid file corruption during save Ahmed Nematallah 2026-05-03 05:12:21 +02:00
  • 97fd047aab Implement existing cell check for layout generation Ahmed Nematallah 2026-05-01 22:54:33 +02:00
  • df4ec3ab32 Corrects an error that can cause "extresist" to segfault. This is due to moving away from using the node "location" in favor of a drivepoint specified by a port or other connection point. The "location" record can end up at the default value of "infinity", and should not be used to set the area for searches. This fix is in response to issue #502 on the github issue tracker, submitted by Julian Schwarz. R. Timothy Edwards 2026-05-06 10:21:35 -04:00
  • d0e38df6bf Fixed an error introduced in the last commit (which has been in the code now for 10 days). The last commit fixed an error when using drcCanonicalMaxwidth() multiple times in DRC rule checks on a single edge. At the time, I also added a method to save some drcCanonicalMaxwidth() results to prevent needlessly re-doing the same (potentially expensive) computation over and over. However, where I reused the cached result, I did not check if the number of entries is zero. Other code does not check the number of entries and depends on the prior behavior that drcCanonicalMaxwidth() returns NULL when there are no entries. So the code needed to check if the number of entries is zero before re-using the cached result. Without the fix, any time the caching is used, the last computed maximum area rectangle will be made active even when the current check area does not meet the area requirement, resulting in many false positive errors. R. Timothy Edwards 2026-04-24 11:21:15 -04:00
  • d157eea7f3 Fixed an error that was discovered with the drcCanonicalMaxwidth() routine, which in turn affects various DRC rules like maxrect, widespacing, and runlength spacing. drcTile() was computing the drcCanonicalMaxwidth() result for the tile and reusing it, failing to account for the fact that within the loop over DRCCookies, other rules might require calling drcCanonicalMaxwidth() on a neighboring tile, or on the same tile with a different width requirement. Implemented a cached version, in which three results are kept: One for neighboring tiles (which can never be reused on the same edge), one for the first required use of the routine for the tile, and one for any other use required for the tile. If there are one or two such rules for an edge, then the routine will work at maximum efficiency. If there are three rules, then one will always be a cache hit and reduce the total amount of computation, although it will still be doing a massive amount of redundant computation. If this seems to be something that occurs regularly, then it can be revisited. The existing implementation works fine for all the open PDKs. Some more advanced PDKs with a number of staged wide-spacing rules could have issues. 8.3.637 R. Timothy Edwards 2026-04-14 17:37:10 -04:00
  • 67c6ed9395 Corrected the handling of toolkit callbacks created by the "magic::add_dependency" procedure. The "check" function should always be called first, followed by the dependencies. Because otherwise, if an incorrect value is entered for a device parameter, then it gets used to calculate dependent values *before* it gets corrected by the "check" procedure, resulting in an incorrect value being displayed for the dependent value, even after the bad entry has been corrected. 8.3.636 R. Timothy Edwards 2026-04-13 15:29:51 -04:00
  • f15f0dabbb Added HTML documentation for the "macro" command, after finding that the exiting documentation does not include the handful of options available to the macro command, nor does it explain the relationship between tools and macro sets. R. Timothy Edwards 2026-04-13 13:01:54 -04:00
  • ae6d26578e Fixed issues with the "macro" command that have existed since the "tool" command was changed to be overridden by a Tcl command. Due to macro clients being registered with the tool name instead of just "layout", the "macro" command with no arguments or with a window client argument was just broken. In the process of fixing this, I realized that there was a conflict between the use of "netlist" as a window name and also as the name of a tool, so I changed the tool name to "nettool", a change which should be transparent to the end user. Otherwise, "macro netlist" returns the key bindings for the window, and the only way to get the key bindings for the tool is to make the tool active and then use "macro" without arguments. One remaining issue is that there is no syntax error that will cause the list of valid windows and tools to be printed. Probably "macro help" should print usage information instead of acting like "macro" with no arguments. R. Timothy Edwards 2026-04-13 12:26:42 -04:00
  • cb1653b157 Found that I needed to apply the same correction as the last commit to the "tagged" GDS/CIF operator, or else this will have exactly the same issue as "bloat-all". 8.3.635 R. Timothy Edwards 2026-04-10 13:39:17 -04:00
  • a3f5e665d1 Found an extremely bad error, where the "bloat-all" GDS output operator does a second search to reset tiles and clear the "processed" mark. This second search was only searching on the types being bloated *into* and not the original types being bloated. This caused tiles to remain marked as "processed" which could in turn cause the tiles to be ignored forever after, potentially resulting in bad GDS output. 8.3.634 R. Timothy Edwards 2026-04-09 11:13:30 -04:00
  • 2eea849c06 Corrected a crash condition in R-C extraction (extresist) caused by old code that still references devices like FETs assuming four ports. I resolved the crash condition but have not worked on fixing the code to properly handle a variable number of terminals per device. 8.3.633 R. Timothy Edwards 2026-04-07 21:15:56 -04:00
  • 6e295d030e Also updated the HTML command reference for the "expand" and "unexpand" commands to include the options just implemented. 8.3.632 R. Timothy Edwards 2026-04-04 21:03:06 -04:00
  • ceba050a21 Fixed the mess that is the "expand"/"unexpand" command set. This had numerous problems, the main one being that each of the three commands was inconsistent: "expand toggle" inverts the expansion of the selection and syncs with the layout. But "expand" expands the layout where instances overlap the box and does not sync to the selection, and "unexpand" unexpands the layout where the box completely surrounds instances. Added a set of options to "expand" and "unexpand" so that these functions can be made consistent with each other. All varieties of the function now always sync the selection and the layout. R. Timothy Edwards 2026-04-04 20:47:04 -04:00
  • f7cceed5e3 Made a fairly major overhaul to the toolkit for generated devices. The main changes are: (1) Changed the hash function used to generate the 6-character suffix for generated device cell names. The original hash function is not good for ensuring unique names, and can cause cell name collisions (two different parameter sets having the same cell name). The chance of name collisions should now be diminishingly small. (2) Modified the string passed to the hash: (A) Ordered the parameter names alphabetically, since iterating through dictionary keys is not guaranteed to be in any specific order, leading to different strings for the same parameter set (B) Normalized numerical parameters, so that "2", "2.0", "2e0" are all hashed the same, again to avoid having multiple cell names for the same set of parameters. (3) Fixed a problem in which when changing parameters for a cell instance, the instance would become unselected and the instance name would be lost and revert to magic's auto-generated name. (4) Fixed the annoyance of having a pop-up dialog whenever magic decides that a parameterized cell name is not being used anywhere, and it can safely delete the cell. (5) Fixed an issue where the check for whether a cell can be deleted is not run consistently. The result is, I hope, a much more pleasant experience with generated cells. 8.3.631 R. Timothy Edwards 2026-04-02 17:13:17 -04:00
  • 5d35ae38b5 Corrected an issue causing MASKHINTS values to fail to be written back to the file, instead resulting in a property key only and no value; this in turn produces an error when read back in. Another cycle of reading and writing causes the property to disappear altogether. R. Timothy Edwards 2026-04-02 14:45:08 -04:00
  • 05561b90f3 Solved three issues: (1) There was still one place that dimensional units were not being printed, which was "setlabel", with both "setlabel size" and "setlabel offset". This has been fixed. (2) Fixing (1) surfaced an error in the text helper dialog, which was not setting units properly when checking the size and offset entries. (3) Fixed a very long-standing problem in which port labels were showing up with the "pale" style of a subcell's label being drawn on top when the parent and child had the same label in exactly the same position. This was due to a parent-first search resulting in the child cell being visited last, so its "pale" style label being drawn last. Added a flag for label searches for display to reverse the search order, visiting children first and then the parent, so that the top cell's labels are the last to be drawn. 8.3.630 R. Timothy Edwards 2026-04-01 21:17:54 -04:00
  • 06eab7feb6 Removed three lines of code that were meant to be a test of how much performance can be sped up if the DRC process can be interrupted at much finer intervals. Unfortunately, the method cannot work without an additional method to ensure that magic is immediately aware when a search routine has been entered recursively and to then break out of the search and clean up after itself. For now, I am just commenting out the code so that magic isn't unstable. 8.3.629 R. Timothy Edwards 2026-03-26 19:44:00 -04:00
  • 9ade4c931e Updated the version to go along with the merge of two pull requests from Darryl Miles. The second of these should finally put to rest all the compile-time issues related to "termio/termios" (yay). R. Timothy Edwards 2026-03-26 09:23:58 -04:00
  • 1bb5316d8d utils/magsgtty.h removal and fixup Darryl L. Miles 2025-07-31 22:59:00 +01:00
  • 2f26237b8b GHA: main.yml/main-aarch64.yml: WASM build Darryl L. Miles 2026-03-25 12:39:20 +00:00
  • 003506ba62 CmdLQ.c: fix MAGIC_WRAPPER code section(s) for WASM build Darryl L. Miles 2026-03-25 12:39:20 +00:00
  • d659b5e1df textio.h: add include <stdarg.h> due to va_list type use in file Darryl L. Miles 2026-03-25 12:39:20 +00:00
  • a0afe242c8 extract/ExtNghbors.c: fix older compiler support for label Darryl L. Miles 2026-03-23 22:41:59 +00:00
  • 88ca77f6a4 GHA: main.yml: remove vezzal task Darryl L. Miles 2026-03-25 12:39:20 +00:00
  • 9d967bdf64 GHA: canary-matrix.yml: apt-get update Darryl L. Miles 2026-03-23 22:40:51 +00:00
  • 7fd2ef8100 I admit I was juggling with too many balls in the air yesterday and managed to commit some code that I thought I had tested--- but apparently what I tested was not the latest version of the code. So 8.3.627 is quite broken and won't even compile. This update fixes that. 8.3.628 R. Timothy Edwards 2026-03-25 14:07:54 -04:00
  • e789f18523 Yet another pass at the problem caused by implementing DRC exceptions; avoided the use of "signed char" altogether by just making the drcc_exception value a bit mask with the lower 7 bits being the index and the upper bit being the flag for exception (0) vs. exemption (1), with the value 0xff being reserved for "none" (no exception or exemption). This implementation should avoid any issues caused by ambiguity in the use of signed vs. unsigned char. 8.3.627 R. Timothy Edwards 2026-03-24 09:37:50 -04:00
  • 460a357730 Corrected what appears to be a long-standing error in the GDS read routine, in which it calls the same "cleanup" routine as for CIF files, which in turn processes the top level cell. However, GDS doesn't have the same concept of a top level cell, so it ends up re-processing part of the last cell read. This would appear to be harmless, except that there is some kind of scaling discrepancy in which the cell bounding box can get incorrectly scaled by the cleanup routine. The solution appears to be simply (?) to not do the "top level" processing. Bypassing the CIFPaintCurrent() from the cleanup routine when reading a GDS file appears to solve the problem. 8.3.626 R. Timothy Edwards 2026-03-22 13:11:27 -04:00
  • 7ceeccef56 Corrected an issue found by Harald Pretl and solved by Claude. Use of "char" does *not* implicitly mean "signed" on ARM architectures. R. Timothy Edwards 2026-03-22 11:14:19 -04:00
  • 66faf1d907 Added documentation for the attribute labels in the "label" command description, because otherwise it only appears in the tech file reference, where it doesn't really belong since the labels are a user-level feature, even if they do affect extraction. The new feature of using gate attributes for parameters was added to the command description. I noticed that the tech file reference made mention of a gate label in the form of ":ext <name>=<value>", but this seems not to be implemented anywhere. At any rate, the method just committed supercedes that. 8.3.625 R. Timothy Edwards 2026-03-18 19:28:23 -04:00
  • 37db9e453b Extended the use of attribute labels to allow gate attributes to specify additional parameters for a device. Any gate attribute (label ending with "^") that has an "=" in it is considered a parameter, and is output with the device in the .ext file. This gives a way to add specific parameters to a device, which otherwise have no meaning to the layout editor. Note that prior to this extension, the gate attribute was used only to set the device index. If a gate attribute is made for a device index, it cannot have "=" in the name, so it remains backwardly compatible. Only non-parameter attributes are passed on to ext2spice. R. Timothy Edwards 2026-03-18 16:41:42 -04:00
  • 2d5c4be6dd Added a few lines of code to the toolkit management to work around the issue of a device name in a netlist having different case than the procedures defined for the device in the PDK. The difference in case is transparent to SPICE, so it should also be transparent to the PDK. R. Timothy Edwards 2026-03-18 12:18:42 -04:00
  • a9673e45ae Added inductors to the list of devices for which the toolkit can call a device generator for an ideal component in a netlist. 8.3.624 R. Timothy Edwards 2026-03-17 11:42:16 -04:00
  • 8f95efc257 Added a feature to the toolkit scripts for generated devices to allow ideal devices in an input netlist to be mapped to automatically generated cells. This works if the PDK toolkit defines a device called either "capacitor" or "resistor". The device must be defined such that it can determine sane parameters for the drawn device from only a value given in the netlist (capacitance in farads, or resistance in ohms). R. Timothy Edwards 2026-03-17 11:32:53 -04:00
  • fd12c39c37 Modified the toolkit scripts so that the "add_dependency" procedure actually works, and also can be called multiple times. The chaining of commands and bindings had earlier been done for one of the widget types (selection) but not the others (entry or checkbox). 8.3.623 R. Timothy Edwards 2026-03-16 20:02:09 -04:00
  • 7d9167257a Corrected an error caused by a recent commit that was attempting to avoid bogus values for array pitch in the direction in which a cell is not arrayed: If the array declares an ANGLE of 90 or 270, then rows and columns are effectively swapped for the purpose of figuring out which coordinate direction should be ignored in the input. The original example did not use ANGLE and so the exception was not handled. R. Timothy Edwards 2026-03-16 12:50:17 -04:00
  • 058b320c3f Also applied the same correction as the previous commit, to hierarchical extraction (as opposed to flat extraction, which is what the previous commit fixed). 8.3.621 R. Timothy Edwards 2026-03-15 15:55:48 -04:00
  • fee4b887c0 Corrected an issue in ext2spice in which devices extracted as ideal, unmodeled capacitors or resistors are missing a space between the 2nd terminal and the device value in the output netlist. This affects the ability to produce correct output for, for example, a metal fingered capacitor using the "device" property to generate an ideal capacitor in the netlist. R. Timothy Edwards 2026-03-15 15:40:10 -04:00
  • 04e91d640d Corrected a problem in which if a label exists that is only connected to material in a child cell, and the label is a point or line label, and the parent cell has no material other than the label in the immediate area, then the "interaction area" will have zero area, which will cause problems as area checks on a zero-sized area don't do anything useful. The solution is to ensure that interaction areas do not have zero area, but will expand around point labels. This should not have any other implications, because the interaction area is just a way to limit the amount of processing; larger interaction areas do not affect the extraction result (except in this case, where it fixes the error). 8.3.620 R. Timothy Edwards 2026-03-12 11:50:21 -04:00
  • 7d9210a3f1 Updated the HTML reference for the "property" command, which did not describe the "list" option (which was just as well, since the option didn't work). 8.3.619 R. Timothy Edwards 2026-03-11 16:43:29 -04:00
  • 03bbc544b2 Fixed a long-standing error that had gone unnoticed that prevents the use of "property list <key>" to return NULL if <key> is undefined, instead of printing an error message that cannot be suppressed. Scripts which wish to test whether or not a bounding box exists will of course want the "quiet" version of the command. R. Timothy Edwards 2026-03-11 16:35:52 -04:00
  • ee79bba5e4 Decided to change the name of the new cifinput/cifoutput operator from "labeled" to "tagged", since there is already an operator called "labels", which is confusing. R. Timothy Edwards 2026-03-11 15:52:42 -04:00
  • 15943d0cb1 Implemented new cifinput/cifoutput operator "labeled" which operates like "bloat-all" except starting with a label and expanding into a type rather than starting with a layer. This is equivalent to what many tools refer to as "stamping". R. Timothy Edwards 2026-03-11 15:06:24 -04:00
  • 7bdd9e1d4f Reworked the method from the previous set of commits, which creates yet another property type "plane". This allows mask hints to be handled like tile planes. That is, after all, what they represent. Although this change is transparent to the end user (apart from a slight performance improvement that is probably not noticeable), it allows for a number of useful future extensions, including the ability to handle non-Manhattan geometry in mask hints, and the ability to view and edit mask hints like any other layout. 8.3.618 R. Timothy Edwards 2026-03-10 14:25:02 -04:00
  • 725c8e9235 Made a few corrections to yesterday's code. Most are minor, but one can potentially cause a crash condition. 8.3.617 R. Timothy Edwards 2026-03-09 14:40:53 -04:00
  • 73ffad3802 Implemented the new feature discussed in the github discussion #492. This features allows two new instructions in the tech file "drc" section: "exception <name>|none" and "exemption <name>|none". The purpose of these rules is to allow DRC rule exceptions for certain areas which are marked, usually by a GDS identifier layer. Because GDS identifier layers can be cast as "mask-hint" properties, the DRC rule exceptions make use of that (recent) feature. So rules can be specified as "exceptions" (rule applies for layout inside the marked area) or "exemptions" (rule is ignored for layout inside the marked area). Rules following the statements will have the exception or exemption applied until another exception or exemption statement is given. Either "exception none" or "exemption none" will stop applying any rule exception or exemption. This is especially useful for applications like SRAM where there may be a number of rule exceptions on different layers. The previous way to handle that was to create new layers in magic for any layer that needed a rule exception. That mainly resulted in very messy tech files, and a large number of defined layers. This implementation is both simpler and cleaner. 8.3.616 R. Timothy Edwards 2026-03-08 20:12:58 -04:00
  • 037daf1121 Found an issue with coupling capacitance where contact types not surrounded by metal are not checked for perimeter coupling parasitic capacitance to other layers. 8.3.615 R. Timothy Edwards 2026-03-05 18:04:53 -05:00
  • d0ef32de0f Tracked down a DRC issue found recently where spurious errors get drawn outside the area being worked on. Tracked it down to a missing check for an invalid rectangle. Fixed now. R. Timothy Edwards 2026-03-05 10:17:29 -05:00
  • 0d9b862c0e Corrected a minor error in ext2hier.c where views with no devices and no ports are still supposed to be output as instances if they have been marked as abstract views. Such views were getting output as cell definitions but not instanced, which although is irrelevant for simulation, would potentially cause LVS errors. R. Timothy Edwards 2026-03-05 09:35:28 -05:00
  • 71dc472797 Modified the toolkit code to make use of the "units" command; otherwise, the existing code would produce results that vary by units, with potentially incorrect label sizes and spacing between instances when generating layout devices and subcells from a netlist. 8.3.614 R. Timothy Edwards 2026-03-04 13:34:31 -05:00
  • 2929ef583e Minor correction to the new "repeat <steps> ... endrepeat" function for tech files, as the first implementation was repeating one too many times. Also: The wrong type was being read for the number of steps, although the resulting value was correct. 8.3.613 R. Timothy Edwards 2026-03-02 12:24:46 -05:00
  • ecd6ec56ae Modified the routine for the cif-output "bloat-all <dist>" operator to make it slightly more efficient. Noting, however, that it is still highly inefficient relative to the stepped "grow" + "and" (or "and-not") operators that it was supposed to replace, I also implemented a convenience function "repeat <steps>" ... "endrepeat" to the tech file syntax. This allows the "grow" + "and" series to be defined with a few lines, whereas if the steps have to be spelled out, the series can be scores or hundreds of lines long. 8.3.612 R. Timothy Edwards 2026-03-01 21:35:00 -05:00
  • f3478cba7b Fixed a few issues related to the "save <file>.tcl": (1) Dimension properties were not output correctly. (2) Magic insisted that the output file was <cell>.tcl, not <file>.tcl, although the error was only in the messaging (3) Magic was incorrectly renaming the cell to <cell>.tcl after writing the file. R. Timothy Edwards 2026-03-01 11:29:25 -05:00
  • ba5154698d Thought better of the recent change to make "snap" no longer change units; although that is a more proper usage, it also happens to break a lot of existing scripts. Since virtually all of these scripts start out with "snap internal" to force units, I have kept this single usage, when "units" is still set to "default". That keeps the older scripts from breaking, while hopefully scripts get updated over time to use "units" instead of "snap". Note that there are still ways to make this behavior break, such as running "units microns" to set units and then sourcing a script that assumes that "snap internal" will change the units, which will then not happen. But batch scripts should be handled property with backwards- compatibility mode. 8.3.611 R. Timothy Edwards 2026-02-28 20:07:00 -05:00
  • 1023461ca5 Minor change to the "save <file>.tcl" method to only paint from TECHDEPBASE, as the layers below that cannot be directly painted; this avoids putting things like "paint checkpaint" into the output, which are harmless but useless. 8.3.610 R. Timothy Edwards 2026-02-27 12:08:35 -05:00
  • 197763e46e Modified the output of "save <cell>.tcl" so that (1) it strips the ".tcl" off of the name, so that the cell name remains the same, and (2) it uses "units" instead of "snap", as the latter is no longer used for setting the unit type. R. Timothy Edwards 2026-02-27 11:18:27 -05:00