sb0 changed the topic of #m-labs to: https://m-labs.hk :: Mattermost https://chat.m-labs.hk :: Logs http://irclog.whitequark.org/m-labs
<_whitenotifier-3> [nmigen] mithro commented on issue #134: Add FSM state name annotations to generated Verilog output - https://git.io/fji4r
<cr1901_modern> whitequark: Re: SPIResource name and dir, is it as simple as adding those as parameters to SPIResource function?
<whitequark> yeah
<cr1901_modern> I'll make a PR right from the GH editor if you can clean it up/review later tonight. I'm going to bed after this.
<_whitenotifier-3> [nmigen-boards] cr1901 opened pull request #20: dev.spi: Update SPIResources to accept a name and direction. - https://git.io/fji4p
<cr1901_modern> whitequark: If you absolutely think it's unsalvageable, just close it or completely rewrite it. At least the PR is open now.
<cr1901_modern> Bedtime for me
<_whitenotifier-3> [nmigen-boards] mithro reviewed pull request #20 commit - https://git.io/fjiBJ
<_whitenotifier-3> [nmigen-boards] whitequark reviewed pull request #20 commit - https://git.io/fjiBI
<_whitenotifier-3> [nmigen-boards] whitequark reviewed pull request #20 commit - https://git.io/fjiBL
rohitksingh_work has joined #m-labs
tweakoz has joined #m-labs
rohitksingh_wor1 has joined #m-labs
rohitksingh_work has quit [Ping timeout: 268 seconds]
_whitelogger has joined #m-labs
tweakoz has quit [Quit: My MacBook has gone to sleep. ZZZzzz…]
cr1901_modern has quit [Ping timeout: 248 seconds]
cr1901_modern has joined #m-labs
Getorix has quit [Ping timeout: 268 seconds]
m4ssi has joined #m-labs
<_whitenotifier-3> [m-labs/nmigen] whitequark pushed 2 commits to master [+0/-0/±2] https://git.io/fjiEo
<_whitenotifier-3> [m-labs/nmigen] whitequark 2492f49 - back.rtlil: use a more principled approach to attributes. NFC.
<_whitenotifier-3> [m-labs/nmigen] whitequark 70f3563 - back.rtlil: attach source locations to switches, not processes.
<_whitenotifier-3> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/555674211?utm_source=github_status&utm_medium=notification
<_whitenotifier-3> [nmigen] Failure. 80.68% (-0.07%) compared to 31c54d3 - https://codecov.io/gh/m-labs/nmigen/commit/70f3563b5fd101668570e7851876b3d3e7b8f347
<_whitenotifier-3> [nmigen] Failure. 42.85% of diff hit (target 80.74%) - https://codecov.io/gh/m-labs/nmigen/commit/70f3563b5fd101668570e7851876b3d3e7b8f347
<_whitenotifier-3> [m-labs/nmigen] whitequark pushed 4 commits to master [+0/-0/±8] https://git.io/fjiuW
<_whitenotifier-3> [m-labs/nmigen] whitequark dac6275 - hdl.ast: use keyword-only arguments as appropriate.
<_whitenotifier-3> [m-labs/nmigen] whitequark 710a8d0 - back.rtlil: ignore empty source locations.
<_whitenotifier-3> [m-labs/nmigen] whitequark 8c9fdf9 - hdl.{dsl,mem,xfrm}: inject appropriate source locations.
<_whitenotifier-3> [m-labs/nmigen] whitequark ec7fcd3 - hdl.xfrm: don't overwrite source locations on ClockDomain signals.
<_whitenotifier-3> [nmigen] Failure. The Travis CI build failed - https://travis-ci.org/m-labs/nmigen/builds/555692919?utm_source=github_status&utm_medium=notification
Getorix has joined #m-labs
<_whitenotifier-3> [m-labs/nmigen] whitequark pushed 3 commits to master [+1/-0/±3] https://git.io/fjiuo
<_whitenotifier-3> [m-labs/nmigen] whitequark 447bfa6 - compat.genlib.fsm: fix after commit dac62754.
<_whitenotifier-3> [m-labs/nmigen] whitequark c14d074 - examples/basic/ctr_ce: fix outdated syntax.
<_whitenotifier-3> [m-labs/nmigen] whitequark 5c63177 - test: generate examples to verilog as part of unit tests.
Getorix has quit [Ping timeout: 248 seconds]
<_whitenotifier-3> [nmigen] Failure. The Travis CI build failed - https://travis-ci.org/m-labs/nmigen/builds/555698233?utm_source=github_status&utm_medium=notification
<_whitenotifier-3> [m-labs/nmigen] whitequark pushed 1 commit to master [+0/-0/±1] https://git.io/fjiu1
<_whitenotifier-3> [m-labs/nmigen] whitequark fd20e17 - test: fix Travis.
<_whitenotifier-3> [nmigen] Failure. The Travis CI build failed - https://travis-ci.org/m-labs/nmigen/builds/555701430?utm_source=github_status&utm_medium=notification
<_whitenotifier-3> [m-labs/nmigen] whitequark pushed 1 commit to master [+0/-0/±1] https://git.io/fjiuM
<_whitenotifier-3> [m-labs/nmigen] whitequark 345a26b - test: fix Travis.
<_whitenotifier-3> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/555702033?utm_source=github_status&utm_medium=notification
<_whitenotifier-3> [nmigen] Success. 80.76% remains the same compared to 5c63177 - https://codecov.io/gh/m-labs/nmigen/commit/345a26b04b577e4580c3b63d4d239b0468903eb2
<_whitenotifier-3> [nmigen] Success. Coverage not affected when comparing 5c63177...345a26b - https://codecov.io/gh/m-labs/nmigen/commit/345a26b04b577e4580c3b63d4d239b0468903eb2
<_whitenotifier-3> [nmigen-boards] cr1901 reviewed pull request #20 commit - https://git.io/fjize
<_whitenotifier-3> [nmigen-boards] cr1901 reviewed pull request #20 commit - https://git.io/fjizf
<_whitenotifier-3> [m-labs/nmigen] whitequark pushed 4 commits to master [+0/-0/±14] https://git.io/fjizJ
<_whitenotifier-3> [m-labs/nmigen] whitequark a7fbff9 - hdl.{ast,cd,dsl,xfrm}: reject inappropriately used comb domain.
<_whitenotifier-3> [m-labs/nmigen] whitequark f0c1c2c - build.dsl: allow assertions on subsignal widths.
<_whitenotifier-3> [m-labs/nmigen] whitequark 0b844da - build.{dsl,res}: allow removing attributes from subsignals.
<_whitenotifier-3> [m-labs/nmigen] whitequark bfbeca4 - back.rtlil: don't name-prefix signals connected to instance ports.
<_whitenotifier-3> [nmigen] whitequark closed issue #125: "comb" should not be accepted as a name in ClockDomain(), DomainRenamer(), etc - https://git.io/fj6fe
<_whitenotifier-3> [nmigen] whitequark closed issue #128: Allow removing attributes in subsignals - https://git.io/fj6dr
<_whitenotifier-3> [nmigen] whitequark closed issue #129: Allow asserting on subsignal widths - https://git.io/fj6dK
<_whitenotifier-3> [nmigen-boards] whitequark opened issue #21: Refactor awkward use of attributes - https://git.io/fjizT
<_whitenotifier-3> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/555718668?utm_source=github_status&utm_medium=notification
<_whitenotifier-3> [nmigen] Success. 80.85% (+0.08%) compared to 345a26b - https://codecov.io/gh/m-labs/nmigen/commit/bfbeca4584f18b2827db157ad43b0d5075cfcacf
<_whitenotifier-3> [nmigen] Success. 100% of diff hit (target 80.76%) - https://codecov.io/gh/m-labs/nmigen/commit/bfbeca4584f18b2827db157ad43b0d5075cfcacf
<cr1901_modern> Oh so assert_width is specifically meant for those *Resource functions where the user is passing in pins
<whitequark> es
<whitequark> *yes
<_whitenotifier-3> [m-labs/nmigen] whitequark pushed 1 commit to master [+0/-0/±2] https://git.io/fjizZ
<_whitenotifier-3> [m-labs/nmigen] whitequark 0ab0a74 - hdl.rec: respect modifications to signals in Record.like().
<_whitenotifier-3> [nmigen] whitequark closed issue #126: Record.like ignores modifications to signals - https://git.io/fj6fV
<cr1901_modern> I guess if I had two words to describe nmigen, it would be "robustness" and "consistency"
<_whitenotifier-3> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/555722112?utm_source=github_status&utm_medium=notification
<_whitenotifier-3> [nmigen] Success. Absolute coverage decreased by -<.01% but relative coverage increased by +3.76% compared to bfbeca4 - https://codecov.io/gh/m-labs/nmigen/commit/0ab0a74ec1155857f4a4764c898cfdd8a33074f3
<_whitenotifier-3> [nmigen] Success. 84.61% of diff hit (target 80.85%) - https://codecov.io/gh/m-labs/nmigen/commit/0ab0a74ec1155857f4a4764c898cfdd8a33074f3
<cr1901_modern> I'll add assert_width to SPIResource if you haven't done so later- running behind right now
<whitequark> >I guess if I had two words to describe nmigen, it would be "robustness" and "consistency"
<whitequark> yes.
<whitequark> exactly the things you want in a HDL.
<cr1901_modern> Robustness is that nmigen will not silently "try to do the right thing" if the input doesn't look right. Consistency in how you use the language primitives nmigen supplies using Context Managers to create something greater than the sum of it's parts.
* cr1901_modern will steal this for a blog post^ :P
<_whitenotifier-3> [m-labs/nmigen] whitequark pushed 1 commit to master [+0/-0/±4] https://git.io/fjizi
<_whitenotifier-3> [m-labs/nmigen] whitequark 7b4fbf8 - build.{dsl,res}: allow platform-dependent attributes using callables.
<_whitenotifier-3> [nmigen] whitequark closed issue #132: Allow dependent platform attributes - https://git.io/fjitc
<_whitenotifier-3> [nmigen-boards] whitequark commented on issue #21: Refactor awkward use of attributes - https://git.io/fjizP
<_whitenotifier-3> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/555727519?utm_source=github_status&utm_medium=notification
<_whitenotifier-3> [nmigen] Failure. 80.74% (-0.11%) compared to 0ab0a74 - https://codecov.io/gh/m-labs/nmigen/commit/7b4fbf8e01f0c982d9a33e5acc063de5ddc1be35
<_whitenotifier-3> [nmigen] Failure. 38.46% of diff hit (target 80.85%) - https://codecov.io/gh/m-labs/nmigen/commit/7b4fbf8e01f0c982d9a33e5acc063de5ddc1be35
Getorix has joined #m-labs
rohitksingh_wor1 has quit [Read error: Connection reset by peer]
zng has quit [Quit: ZNC 1.7.2 - https://znc.in]
zng has joined #m-labs
<mithro> whitequark: I think one of the best features of nmigen is the promise of decent error messages....
<whitequark> promise?
<mithro> whitequark: I guess and delivering on that promise :-)
<whitequark> okay
<mithro> whitequark: Maybe it would have been better to say "I think one of the best features of nmigen is the commitment to provide decent error messages...."?
<whitequark> yeah :)
m4ssi has quit [Remote host closed the connection]
<_whitenotifier-3> [nmigen-boards] cr1901 reviewed pull request #19 commit - https://git.io/fjioM
<_whitenotifier-3> [nmigen-boards] cr1901 synchronize pull request #20: dev.spi: Update SPIResources to accept a name and direction. - https://git.io/fji4p
_whitelogger has joined #m-labs
<_whitenotifier-3> [nmigen-boards] Fatsie synchronize pull request #15: Digilent Atlys spartan6 board - https://git.io/fj61v
mumptai has joined #m-labs
<_whitenotifier-3> [nmigen-boards] Fatsie commented on pull request #15: Digilent Atlys spartan6 board - https://git.io/fjiir
mumptai has quit [Quit: Verlassend]
lkcl has quit [Ping timeout: 268 seconds]