whitequark[cis] changed the topic of #amaranth-lang to: Amaranth hardware definition language · weekly meetings: Amaranth each Mon 1700 UTC, Amaranth SoC each Fri 1700 UTC · code https://github.com/amaranth-lang · logs https://libera.irclog.whitequark.org/amaranth-lang · Matrix #amaranth-lang:matrix.org
lf_ has quit [Ping timeout: 260 seconds]
lf has joined #amaranth-lang
polymerizedsage has quit [Quit: Client closed]
RowanG[m] has joined #amaranth-lang
<RowanG[m]> Hi everyone.
<RowanG[m]> I was looking into CRC engine implementation and really liked the one amaranth has: https://github.com/adamgreig/amaranth/blob/main/amaranth/lib/crc/__init__.py
<RowanG[m]> I want to implement one myself as part of clash(A different HDL) of clash cores project. Since I will probably roughly follow the outline in amaranth I would like to check in with you guys if have any problems with me releasing the clash implementation under public domain? Clash itself is completely different from Amaranth.
jjsuperpower has quit [Remote host closed the connection]
jjsuperpower has joined #amaranth-lang
<adamgreig[m]> clash itself is bsd2, same as amaranth (and the amaranth crc impl), is that an option?
<adamgreig[m]> though perhaps your version ends up being different enough that it's not really relevant anyway
<RowanG[m]> Implementation wise it will probably be different. Python and Haskell are pretty different in terms of how you implement things.
<RowanG[m]> I could use BSD2 and add a copyright line for Amaranth?
<RowanG[m]> If that's oke I will do it like that
<adamgreig[m]> that's undoubtably ok, the bsd licence definitely lets you do that, but it's probably overkill if all you take from the amaranth version is the general algorithm, then I think just a link back in a comment would be kind
notgull has joined #amaranth-lang
peeps has joined #amaranth-lang
peeps[zen] has quit [Read error: Connection reset by peer]
notgull has quit [Ping timeout: 264 seconds]
ebb has quit [Remote host closed the connection]
ebb has joined #amaranth-lang
peeps has quit [Remote host closed the connection]
peeps has joined #amaranth-lang
Degi has quit [Ping timeout: 255 seconds]
Degi has joined #amaranth-lang
Xesxen_ is now known as Xesxen
jjsuperpower has quit [Ping timeout: 268 seconds]
<Wanda[cis]> Catherine: I think the `yowasp-yosys` requirement in `pyproject.toml` should be changed to `>=0.38` to match the check in the backend? the recent changes broke my environment since pdm just reused a 0.34 I had laying around and I spent a while debugging what exactly went wrong...
<_whitenotifier-5> [amaranth] wanda-phi opened pull request #1140: Implement RFC 43: Rename `reset=` to `init=`. - https://github.com/amaranth-lang/amaranth/pull/1140
<_whitenotifier-5> [amaranth] codecov[bot] commented on pull request #1140: Implement RFC 43: Rename `reset=` to `init=`. - https://github.com/amaranth-lang/amaranth/pull/1140#issuecomment-1943146924
<_whitenotifier-5> [amaranth] wanda-phi opened pull request #1141: pyproject: change yosys-yowasp requirement to match RTLIL backend. - https://github.com/amaranth-lang/amaranth/pull/1141
<_whitenotifier-7> [amaranth] codecov[bot] commented on pull request #1141: pyproject: change yosys-yowasp requirement to match RTLIL backend. - https://github.com/amaranth-lang/amaranth/pull/1141#issuecomment-1943152704
byteit101 has quit [Ping timeout: 268 seconds]
byteit101 has joined #amaranth-lang
<Wanda[cis]> annoying RFC 45 consequence: easy to shadow amaranth.lib.memory.Memory with the old amaranth.hdl.mem.Memory via a star import from amaranth
<galibert[m]> don't put the deprecated one in the star import list?
<Wanda[cis]> that's an immediate breaking change
<galibert[m]> there's going to be something breaking at some point given the name is the same
<_whitenotifier-7> [amaranth] wanda-phi opened pull request #1142: Implement RFC 45: Move `hdl.Memory` to `lib.Memory`. - https://github.com/amaranth-lang/amaranth/pull/1142
<_whitenotifier-5> [amaranth] codecov[bot] commented on pull request #1142: Implement RFC 45: Move `hdl.Memory` to `lib.Memory`. - https://github.com/amaranth-lang/amaranth/pull/1142#issuecomment-1943482720
<_whitenotifier-5> [amaranth] whitequark closed pull request #1138: sim: fix using 0-width `Switch`. [0.4 backport] - https://github.com/amaranth-lang/amaranth/pull/1138
<_whitenotifier-5> [amaranth] whitequark reviewed pull request #1141 commit - https://github.com/amaranth-lang/amaranth/pull/1141#discussion_r1489350321
<_whitenotifier-7> [amaranth] github-merge-queue[bot] created branch gh-readonly-queue/main/pr-1141-353a8ce7e313e8d203635151402ef88a0171bf45 - https://github.com/amaranth-lang/amaranth
<_whitenotifier-7> [amaranth] github-merge-queue[bot] created branch gh-readonly-queue/main/pr-1136-8033ddf05ed0a4723d2449b640263df040d5a12f - https://github.com/amaranth-lang/amaranth
<_whitenotifier-5> [amaranth] github-merge-queue[bot] created branch gh-readonly-queue/main/pr-1134-0ecd06a7e5cac77940c7deb4c0bcd5dbf34869c4 - https://github.com/amaranth-lang/amaranth
<_whitenotifier-5> [amaranth-lang/amaranth] wanda-phi 8033ddf - pyproject: change yosys-yowasp requirement to match RTLIL backend.
<_whitenotifier-7> [amaranth-lang/amaranth] github-merge-queue[bot] pushed 2 commits to main [+0/-0/±3] https://github.com/amaranth-lang/amaranth/compare/353a8ce7e313...0ecd06a7e5ca
<_whitenotifier-5> [amaranth-lang/amaranth] wanda-phi 0ecd06a - sim: fix using 0-width `Switch`.
<_whitenotifier-5> [amaranth] whitequark closed pull request #1136: sim: fix using 0-width `Switch`. - https://github.com/amaranth-lang/amaranth/pull/1136
<_whitenotifier-7> [amaranth] github-merge-queue[bot] deleted branch gh-readonly-queue/main/pr-1141-353a8ce7e313e8d203635151402ef88a0171bf45 - https://github.com/amaranth-lang/amaranth
<_whitenotifier-5> [amaranth] github-merge-queue[bot] deleted branch gh-readonly-queue/main/pr-1136-8033ddf05ed0a4723d2449b640263df040d5a12f - https://github.com/amaranth-lang/amaranth
<_whitenotifier-5> [amaranth] whitequark closed pull request #1141: pyproject: change yosys-yowasp requirement to match RTLIL backend. - https://github.com/amaranth-lang/amaranth/pull/1141
<_whitenotifier-5> [amaranth-lang/amaranth-lang.github.io] whitequark pushed 1 commit to main [+0/-0/±34] https://github.com/amaranth-lang/amaranth-lang.github.io/compare/5788005b7f78...a1309cadf3d4
<_whitenotifier-7> [amaranth-lang/amaranth-lang.github.io] github-merge-queue[bot] a1309ca - Deploying to main from @ amaranth-lang/amaranth@0ecd06a7e5cac77940c7deb4c0bcd5dbf34869c4 🚀
<_whitenotifier-5> [amaranth-lang/amaranth] wanda-phi 5ffb48b - hdl._ast: fix using 0-width `Switch` with integer keys.
<_whitenotifier-7> [amaranth-lang/amaranth] github-merge-queue[bot] pushed 1 commit to main [+0/-0/±2] https://github.com/amaranth-lang/amaranth/compare/0ecd06a7e5ca...5ffb48b5fb4a
<_whitenotifier-7> [amaranth] whitequark closed pull request #1134: hdl._ast: fix using 0-width `Switch` with integer keys. - https://github.com/amaranth-lang/amaranth/pull/1134
<_whitenotifier-5> [amaranth] github-merge-queue[bot] deleted branch gh-readonly-queue/main/pr-1134-0ecd06a7e5cac77940c7deb4c0bcd5dbf34869c4 - https://github.com/amaranth-lang/amaranth
<_whitenotifier-5> [amaranth-lang/amaranth-lang.github.io] whitequark pushed 1 commit to main [+0/-0/±34] https://github.com/amaranth-lang/amaranth-lang.github.io/compare/a1309cadf3d4...d9c3527e55ec
<_whitenotifier-7> [amaranth-lang/amaranth-lang.github.io] github-merge-queue[bot] d9c3527 - Deploying to main from @ amaranth-lang/amaranth@5ffb48b5fb4ab08b9587504956a8d52ab75e33cd 🚀
<_whitenotifier-5> [amaranth] whitequark closed pull request #1137: hdl._ast: fix using 0-width `Switch` with integer keys. [0.4 backport] - https://github.com/amaranth-lang/amaranth/pull/1137
electronic_eel has quit [Remote host closed the connection]
electronic_eel has joined #amaranth-lang
electronic_eel has quit [Quit: No Ping reply in 180 seconds.]
electronic_eel has joined #amaranth-lang
<_whitenotifier-7> [amaranth] codecov[bot] commented on pull request #1139: hdl._dsl: fix using 0-width `Switch` with integer keys. [0.4 backport] - https://github.com/amaranth-lang/amaranth/pull/1139#issuecomment-1944093585
<_whitenotifier-5> [amaranth] codecov[bot] commented on pull request #1135: hdl._dsl: fix using 0-width `Switch` with integer keys. - https://github.com/amaranth-lang/amaranth/pull/1135#issuecomment-1944093901
<Wanda[cis]> oops doing a trivial comb loop (m.d.comb += [a.eq(b), b.eq(a)]) hangs NIR builder as it tries to resolve nets
feldim2425 has quit [Quit: ZNC 1.8.2+deb2build5 - https://znc.in]
feldim2425 has joined #amaranth-lang
<_whitenotifier-7> [amaranth] wanda-phi opened issue #1143: infinite loop in NIR builder on a trivial comb loop - https://github.com/amaranth-lang/amaranth/issues/1143
Chips4MakersakaS has quit [Quit: Idle timeout reached: 172800s]
notgull has joined #amaranth-lang
notgull has quit [Ping timeout: 264 seconds]
catlosgay[m] has quit [Quit: Idle timeout reached: 172800s]
<whitequark[cis]> <Wanda[cis]> "oops doing a trivial comb loop (..." <- in a way actually preferable to it completing
<Wanda[cis]> heh
<_whitenotifier-7> [amaranth] whitequark closed pull request #1139: hdl._dsl: fix using 0-width `Switch` with integer keys. [0.4 backport] - https://github.com/amaranth-lang/amaranth/pull/1139
<_whitenotifier-7> [amaranth] github-merge-queue[bot] created branch gh-readonly-queue/main/pr-1135-5ffb48b5fb4ab08b9587504956a8d52ab75e33cd - https://github.com/amaranth-lang/amaranth
<_whitenotifier-7> [amaranth] whitequark closed pull request #1135: hdl._dsl: fix using 0-width `Switch` with integer keys. - https://github.com/amaranth-lang/amaranth/pull/1135
<_whitenotifier-5> [amaranth-lang/amaranth] wanda-phi e3324e1 - hdl._dsl: fix using 0-width `Switch` with integer keys.
<_whitenotifier-7> [amaranth-lang/amaranth] github-merge-queue[bot] pushed 1 commit to main [+0/-0/±2] https://github.com/amaranth-lang/amaranth/compare/5ffb48b5fb4a...e3324e14566f
<_whitenotifier-7> [amaranth] whitequark closed issue #1133: spurious warning for `Switch` over 0-width value - https://github.com/amaranth-lang/amaranth/issues/1133
<_whitenotifier-7> [amaranth] github-merge-queue[bot] deleted branch gh-readonly-queue/main/pr-1135-5ffb48b5fb4ab08b9587504956a8d52ab75e33cd - https://github.com/amaranth-lang/amaranth
<_whitenotifier-5> [amaranth-lang/amaranth-lang.github.io] github-merge-queue[bot] d4b15cb - Deploying to main from @ amaranth-lang/amaranth@e3324e14566f15653c119ec68ec83d71eea0c04c 🚀
<_whitenotifier-7> [amaranth-lang/amaranth-lang.github.io] whitequark pushed 1 commit to main [+0/-0/±34] https://github.com/amaranth-lang/amaranth-lang.github.io/compare/d9c3527e55ec...d4b15cb87958
<Wanda[cis]> okay, I've thrown in some (not very good) docs for lib.memory
<_whitenotifier-5> [amaranth] wanda-phi edited pull request #1142: Implement RFC 45: Move `hdl.Memory` to `lib.Memory`. - https://github.com/amaranth-lang/amaranth/pull/1142
<Wanda[cis]> ... actually, couldn't we just use the Memory itself instead of MemoryIdentity? it seems like anything hashable will do
<whitequark[cis]> didn't we encounter issues with transformers?
<Wanda[cis]> that's for using MemoryInstance as identity
<whitequark[cis]> hm
<whitequark[cis]> I'm confused
<whitequark[cis]> oh, is the idea to basically use any old object as an identity, and leave it user provided?
<whitequark[cis]> yeah, happy with that
<Wanda[cis]> yup
<Wanda[cis]> absolutely no problems with just grabbing any random object as your identity
<whitequark[cis]> sgtm
<whitequark[cis]> this register will be sad if you don't allocate it
mcc111[m] has quit [Quit: Idle timeout reached: 172800s]
Raito_Bezarius has quit [Read error: Connection reset by peer]
Raito_Bezarius has joined #amaranth-lang
<_whitenotifier-7> [amaranth] whitequark commented on issue #1143: infinite loop in NIR builder on a trivial comb loop - https://github.com/amaranth-lang/amaranth/issues/1143#issuecomment-1945010574