UCSBarchlab / PyRTL

A collection of classes providing simple hardware specification, simulation, tracing, and testing suitable for teaching and research. Simplicity, usability, clarity, and extensibility are the overarching goals, rather than performance or optimization.
http://ucsbarchlab.github.io/PyRTL
BSD 3-Clause "New" or "Revised" License
253 stars 76 forks source link

render_trace()'s default symbol_len should be None rather than 5 to e… #424

Closed fdxmw closed 2 years ago

fdxmw commented 2 years ago

render_trace()'s default symbol_len should be None rather than 5 to enable the code that calculates symbol_len from the maximum value length. This should fix issue #423.

codecov-commenter commented 2 years ago

Codecov Report

Merging #424 (8f1bcec) into development (8445b0f) will increase coverage by 0.00%. The diff coverage is n/a.

:exclamation: Current head 8f1bcec differs from pull request most recent head 1d44585. Consider uploading reports for the commit 1d44585 to get more accurate results

@@             Coverage Diff              @@
##           development     #424   +/-   ##
============================================
  Coverage        90.52%   90.52%           
============================================
  Files               24       24           
  Lines             6069     6101   +32     
============================================
+ Hits              5494     5523   +29     
- Misses             575      578    +3     
Impacted Files Coverage Δ
pyrtl/simulation.py 93.43% <ø> (ø)
pyrtl/visualization.py 86.76% <0.00%> (+0.51%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.