FFY00 has quit [Read error: Connection reset by peer]
FFY00 has joined #nmigen
Wolong has joined #nmigen
Degi has quit [Ping timeout: 258 seconds]
Degi has joined #nmigen
cr1901_modern has quit [Read error: Connection reset by peer]
cr1901_modern has joined #nmigen
<cr1901_modern>
whitequark: I won't be at the meeting tomorrow due to rescheduled prior engagement. But hey, that's okay... b/c the thing I wanted to discuss isn't ready yet lmao
<whitequark>
sure
futarisIRCcloud has joined #nmigen
<awygle>
oo awesome wq
<awygle>
thanksf or linking me
electronic_eel has quit [Ping timeout: 258 seconds]
electronic_eel has joined #nmigen
PyroPeter_ has joined #nmigen
PyroPeter has quit [Ping timeout: 260 seconds]
PyroPeter_ is now known as PyroPeter
hitomi2509 has joined #nmigen
jeanthom has joined #nmigen
emeb_mac has quit [Quit: Leaving.]
jeanthom has quit [Ping timeout: 265 seconds]
jeanthom has joined #nmigen
jeanthom has quit [Ping timeout: 264 seconds]
jeanthom has joined #nmigen
Asu has joined #nmigen
lambda has joined #nmigen
jeanthom has quit [Remote host closed the connection]
jeanthom has quit [Remote host closed the connection]
jeanthom has joined #nmigen
<Wolong>
Hello,,I am learning nmigen, and I got an error when running the sample code at this link address (http://blog.lambdaconcept.com/doku.php?id=nmigen:nmigen_hello_world), the error message is as follows:
<Wolong>
$ python3 blinky.py simulate -c 20 -v blinky.vcdTraceback (most recent call last): File "blinky.py", line 19, in <module> main(top, ports=(top.led)) File "/home/fl/.local/lib/python3.7/site-packages/nmigen/cli.py", line 74, in main main_runner(parser, parser.parse_args(), *args, **kwargs) File
<Wolong>
"/home/fl/.local/lib/python3.7/site-packages/nmigen/cli.py", line 68, in main_runner with sim.write_vcd(vcd_file=args.vcd_file, gtkw_file=args.gtkw_file, traces=ports): File "/home/fl/.local/lib/python3.7/site-packages/nmigen/back/pysim.py", line 1117, in write_vcd vcd_file=vcd_file, gtkw_file=gtkw_file, traces=traces) File
<Wolong>
"/home/fl/.local/lib/python3.7/site-packages/nmigen/back/pysim.py", line 95, in __init__ if trace not in signal_names: File "/usr/local/lib/python3.7/_collections_abc.py", line 666, in __contains__ self[key] File "/home/fl/.local/lib/python3.7/site-packages/nmigen/hdl/ast.py", line 1385, in __getitem__ key = None if key is None else
<Wolong>
self._map_key(key) File "/home/fl/.local/lib/python3.7/site-packages/nmigen/hdl/ast.py", line 1575, in __init__ raise TypeError("Object {!r} is not an nMigen signal".format(signal))TypeError: Object (slice (sig led) 0:1) is not an nMigen signal
<Wolong>
Is the nmigen version I installed incompatible with the code here? How should I modify the code to make it work?
<Wolong>
Can someone give me some help information? thank you very much
<agg>
Wolong: there's a bug in that demo code on the final line, where ports=(top.led) is written
<Wolong>
Thanks agg, I solved this problem according to your tips.(y)
chipmuenk has joined #nmigen
jeanthom has quit [Ping timeout: 246 seconds]
hitomi2509 has quit [Quit: Nettalk6 - www.ntalk.de]
emeb has joined #nmigen
<lambda>
hm, should the `tests` module really be installed by `setup.py install`? claiming the entire `import tests` namespace seems... wrong
<agg>
heh, maybe setup.py's packages should be ['nmigen'] instead of find_packages()
danfoster has quit [Ping timeout: 260 seconds]
<lambda>
deleting tests/__init__.py also fixes it - does anything actually require `tests/` to be a module, or can it just be treated as a regular directory?
<agg>
did you try running the tests after removing it? most of the tests use relative imports so i think they'd all break immediately if it wasn't a module
<lambda>
ah, hm, yeah
<lambda>
I thought the import errors I was seeing were just because I didn't actually have the package installed yet
<agg>
actually this is a recent change from aug 27 67b957d4
<agg>
where packages arg in setup.py was changed from find_packages(exclude=["*.test*"]) to just find_packages(), but perhaps wasn't intended to still include the tests top-level package
<agg>
maybe worth opening an issue and tagging that commit, i suspect this isn't intended behaviour
* agg
awaits code with `from tests.utils import FHDLTestCase`