* Bug: In Common.h, makes the INSPECTOR_PV_SUPPORT to generate a getRecord()
by *value* instead of *by reference*. Seems the overload resolution
system has changed and discriminate between them using now the exact
match.
This was causing InspectorWidget::setRootoccurrence() to fail, the
getRecord(Occurrence) not corectly resolved always returning NULL.
* Change: In Occurrence, now use INSPECTOR_PV_SUPPORT (instead of PR).
* Change: In SelectionPopup, added a _cellWidget attribute to be able
to call setShowSelection() in ::mouseReleaseEvent(), same as for
SelectCommand.
* Bug: In SelectionWidget::changeSelectionMode(), reset the _updateState
to ExternalEmit in *all cases* at the end of the function call.
Failing to do that was another cause for the selection check box
to desinchronize.
* Bug: In CellWidget::drawGrid(), the scaling of grid was never recomputed
according to the zoom level. So in low zoom, it was too fine and
hidding everything underneath...
Now always display between 1 and 10 lines. Lines are put on the
grid step.
Makes the grid draw in dotted lines, seems to looks nicer.
* New: In CRL::DefImport, the previous version of the parser was designed
only to read pure netlists, no physical components. Now add features
for:
* VIA generate statements. Generated VIAs are created as Cell and
then instaciated wherever they are needed. Alternative would be
to duplicate it's contents so the original netlist is not changed.
But would create lot more objects.
* PIN, added support for basic physical shapes.
* SPECIALNETS and their associated wiring (path callback).
Note: (to myself) As the Path is created *before* the NET or SPECIALNET
callback is called, we must create a temporary net to store
the path components. This is the "__prebuild__" net which
will be merged later with the actual net.
* Bug: In def/def_keywords.cpp, in defrData::pathIsDone(), the attribute
defrData.PathObj was *forcibly* deleted directly inside the defrData
class (with defrData remaining allocated). Either there is something
I enterily miss or this is a big blunder.
This was causing DefImport parser to crash, when activating the
Path callback.
* Bug: In CellViewer::setShowSelection(bool), this slot is connected
to the hidden 's' menu key. It was toggling state according to
the state of the *menu key* (Qt) which was *not* kept in synch
with the state of the CellWidget. So it was causing desinchronized
selection visibility toggle. Now, ignore the state of the menu
and toggle the CellWidget state directly.
* Bug: In Selector::attachTo(), when adding the new CellWidget to the
map<>, the initial flag must be Selector::Selected instead of 0,
so we see it.
* Change: In CellWidget & ControllerWidget, adjust the default size of
the widgets (to better suit my display...).
* Bug: In Katana::Session::_addMoveEvent(), an error message is issued if
the segment is not *already* in a Track. But, it is not the case for
non-preferred Segment or reduced segment and it is valid. So suppress
the message in those cases.
* Bug: In cumulus/plugins.block.htree._connectLeaf(), when HEAVY_LEAF_LOAD
was used, the additionnal horizontal segment added to connect the
extra buffer may have been:
1. Misaligned, causing the blockage to be wrongly taken into account
and leading to potential short circuits.
2. In some RoutingGauge configurations, the extra length added to
the segment left extention was too short, leading to DRC violations.
So created new GaugeConf methods to manage thoses problems:
* GaugeConf.getStackX() & GaugeConf.getStackY() to get back the
contact coordinates of the *deep* part (typically the METAL2).
So be able to align on them.
* GaugeConf.createHorizontal(), added a parameter to explicitely
state the additional length of the *left* extension. This is lazy,
will soon add both.
* Change: In Hurricane::SharedPath, the hash for the shared pathes, which
serves for Map<> storing, were computed "on the fly" at each
::getHash() call, and were involving recursive calls of all the
hashes along the components of the shared path. This is terribly
slow especially in a design with a deep hierarchy (typical case
LS180).
Now, at the cost of one supplemenental unsigned long in each
SharedPath, it is only computed once in the constructor, and is
no longer recursive (only access the *first* tail, if any).
As a consequence, the re-display computation delay becomes
bearable. But also speed up any transhierarchical walkthough.
* Change: In Etesianengine::toHurricane(), no longer call interTies().
This *should* no longer be needed as almost all standard cells
(SxLib & FlexLib) already contains polarisation contacts.
So do not need to disturb the placement.
Secondly, in presence of a H-Tree, when there is not enough
space to insert a sufficient number of tie cell, the lefmost one
in the sub-row is shifted to left, overlaping the H-Tree buffer
cells. This is a bug (even signaled by a warning). But as this
part is disabled, I won't correct it soon...
* Change: In Cumulus/plugins.block.Block, for standalone block,
the external Pin where created non-square, the "perpandicular"
side made a half wire width. But in case the wire width was an
odd number of physical grid, it was leading to offgrid screaming
in the GDS driver.
Now, just make them square...
* Bug: In CRL::GdsDriver::GdsStream::operator<<(Cell*), when looking for
layer names ending with ".pin", must also check that the string is
at least 4 characters long.
The CMake version set was so old that obsolete policies were being
enabled and breaking FindPython. 2.8 should still be old enough not to
cause problems for any current users.
Signed-off-by: Myrtle Shah <gatecat@chipflow.io>
This field has been deprecated in Python 3 and no longer exists at all
in Python 3.9, causing a build error.
Signed-off-by: Myrtle Shah <gatecat@chipflow.io>
* Bug: In KatanaEngine::annotateGlobalGraph(), the minimum track reservation
was not applied correctly. Instead of adding the *difference* between
the minimum and the current reservation, the minimum was simply added.
Leading to too high reservation. And loss of control about what we
were doing on the edges capacities...
Really stupid.
large enough instead of creating one big via along the horizontal
cell power lines, we create one big VIA at each end. To avoid to
create too massive obstructions.
But the thresold was too high for cmos45, leading to short with
the clock tree trunk.
* Change: In bootstrap/coriolisEnv.py, when all tries to locate the
installation root in predefined locations, try the directory the
script is installed in.
This allows for a user to use an installation made in another
user's account.
* Bug: In Anabatic::_desaturate(), the set of invalidated GCells was never
cleared between iterations of desaturated GCells. This was not a
memory leak per se because, in the end, the set was freed. But it did
induce a gigantic bloating in memory. This finally explain the strange
slow down in that stage.
Detected in ls180+cmos45.
* Change: In Anabatic::GCell, add a flag for each depth to avoid
desaturating twice the same GCell (maybe not needed now that the
bug is corrected, but it ensure a no-looping).
* Bug: In cumulus/plugins.block.configuration.GaugeConf.expandMinArea(),
The minimal length of the segment intermediate wires where computed
for the minimal area using an integer division ( // ), which is
stupid for less than 1.0 values. They are real quantities at this
point...
This was making the DRC of ls180 failing.
Jean-Paul Chaput
Myrtle Shah
Las Safin