Skip to content

Commit

Permalink
Fixes bug when pushin external signals to test vector file that inclu…
Browse files Browse the repository at this point in the history
…ded internals
  • Loading branch information
chaseruskin committed Nov 2, 2024
1 parent c8088af commit 13dd419
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/lib/python/verb/vectors.py
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,9 @@ def push(self, model, ignore_coverage: bool=False):
net: CoverageNet

# ignore the name when collecting the ports for the given mode
signals = [p[1] for p in _extract_ports(model, mode=self._mode)] + [p[1] for p in _extract_signals(model)]
ext_signals = [p[1] for p in _extract_ports(model, mode=self._mode)]
local_signals = [p[1] for p in _extract_signals(model)]
all_signals = ext_signals + local_signals
# check if there are coverages to automatically update
if ignore_coverage == False:
for net in Coverage.get_nets():
Expand All @@ -113,7 +115,7 @@ def push(self, model, ignore_coverage: bool=False):
sinks = net.get_sink_list()
for sink in sinks:
# exit early if a signal being observed is not this transaction
if type(sink) == Signal and sink not in signals:
if type(sink) == Signal and sink not in all_signals:
break
pass
# perform an observation if the signals are in this transaction
Expand All @@ -130,7 +132,7 @@ def push(self, model, ignore_coverage: bool=False):

if self._is_empty == False:
fd.write(NEWLINE)
for info in signals:
for info in ext_signals:
fd.write(str(info) + DELIM)

self._is_empty = False
Expand Down

0 comments on commit 13dd419

Please sign in to comment.