01:00 <cr1901> Finally wrote all the docs for my amaranth simulator wrapper that works with v0.5. This in principle means that I can release it on PyPI right now/tonight! In practice, probably not going to happen due to my executive dysfunction (esp since the Cohost news...)

10:59 <whitequark[cis]> wrapper?

14:44 <cr1901> s/wrapper/pytest fixtures/ ... just things that make it less code to parametrize amaranth sims in pytest

16:43 <mabl[m]> Ah i also have some infrastructure to export signal traces etc. Unfortunately, I have not yet figured out why failing assertions don't give proper failure texts and back trace...

16:43 <mabl[m]> Maybe a pytest thingy

18:40 <trappedinpajamas> hmm, I'm trying to connect a memory read port to another module with an `Out(Readport.Signature())` , but `wiring.connect()` complains with `TypeError: The initial argument must be a module, not <amaranth.lib.memory.ReadPort object at 0x7bc82d5beea0>`. I assume this is an error of my understanding, since AIUI readport meets the requirements for `Instance` and therefore can be used with `connect()`

18:40 <trappedinpajamas> * an `Out(Readport.Signature(...))` ,

18:41 <whitequark[m]> use wiring.connect(m, read_port, other_port)

18:41 <whitequark[m]> not wiring.connect(read_port, other_port)

18:42 <whitequark[m]> by "a module" it means "a `Module` object" (it's unambiguous but I suppose a little confusing)

18:43 <whitequark[m]> we discussed ctx.changed() and expressions and decided it adds too much complexity to the simulator, but I think there is special handling for edge on 1-bit-wide sig[index] that could be extended to changed

18:45 <vegard_e[m]> in theory, is there anything that prevents a wrapper from introspecting the expression and adding triggers on each separate signal?

18:45 <whitequark[m]> we could reword that message to include my clarification in it

18:45 <vegard_e[m]> it'll be glitchy, but combinatorics necessarily is

18:45 <trappedinpajamas> ah, i totally missed that, makes sense.

18:46 <whitequark[m]> nope

18:46 <whitequark[m]> if you file an issue we could improve the error message

18:46 <trappedinpajamas> will do!

19:08 <_whitenotifier-1> [amaranth] 5aji opened issue #1509: Clarify documentation on the module usage in `wiring.connect()` - https://github.com/amaranth-lang/amaranth/issues/1509

19:14 <_whitenotifier-1> [amaranth] whitequark commented on issue #1509: Clarify documentation on the module usage in `wiring.connect()` - https://github.com/amaranth-lang/amaranth/issues/1509#issuecomment-2341827123