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
Stormwind_mobile has joined #m-labs
electronic_eel_ has joined #m-labs
electronic_eel has quit [Ping timeout: 260 seconds]
electronic_eel_ has quit [Ping timeout: 258 seconds]
electronic_eel has joined #m-labs
early has quit [Quit: Leaving]
early has joined #m-labs
airwoodix5 has joined #m-labs
airwoodix has quit [Ping timeout: 240 seconds]
airwoodix5 is now known as airwoodix
Gurty has quit [Ping timeout: 252 seconds]
Gurty has joined #m-labs
cr1901_modern has quit [Ping timeout: 260 seconds]
<zignig> wq: you about ? /
sb0 has joined #m-labs
cr1901_modern has joined #m-labs
<cr1901_modern> sb10q: Right now, migen (and nmigen I think) only supports running the build script in the build directory, correct?
<sb0> cr1901_modern: vivado and ISE dump stuff (e.g. log files) in the current working directory, so changing to the build directory is done to keep those contained
<cr1901_modern> Oh hi, you're on IRC. Well, the cygwin crap should be unnecessary if you don't use absolute paths in the project file _and_ you put all the imported files into the build directory.
<cr1901_modern> Either of those commit to "you can only run the script in the build dir", however
<cr1901_modern> If you don't like either of those, we can just drop Cygwin support. nmigen doesn't support it, and I haven't heard from FelixVi- the person who wanted Cygwin support- in nearly 2 years.
<cr1901_modern> You really do need drive-letter plus path for Xilinx tools, however (if not using relative paths)
<sb0> we want to make the gateware build directory entirely self-contained to simplify the implementation of this https://git.m-labs.hk/M-Labs/nix-scripts/pulls/23
<cr1901_modern> And removing the Cygwin code is required for that PR you linked to work?
<cr1901_modern> Actually I just realized... unless you use relative paths, builds won't be self-contained b/c they encode path information specific to your machine, correct?
<cr1901_modern> cc: Astro-
<sb0> it is not required to remove it, but that PR may render it unnecessary
<sb0> and yes
<cr1901_modern> sb0: I would have to check if the PR renders it unnecessary... my gut feeling is yes. I still think imported code should be in its own subdirectory to avoid pollution of the build dir
<cr1901_modern> well further*
proteusguy has quit [Ping timeout: 268 seconds]
rohitksingh has joined #m-labs
proteusguy has joined #m-labs
<_whitenotifier-5> [nmigen] Fatsie synchronize pull request #296: Signal: allow to use integral Enum for reset value. - https://git.io/JejR0
<_whitenotifier-5> [nmigen] Fatsie commented on pull request #296: Signal: allow to use integral Enum for reset value. - https://git.io/JvvDF
<_whitenotifier-5> [nmigen] Fatsie synchronize pull request #296: Signal: allow to use integral Enum for reset value. - https://git.io/JejR0
<_whitenotifier-5> [nmigen] Fatsie reviewed pull request #296 commit - https://git.io/JvvDN
<_whitenotifier-5> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/635237809?utm_source=github_status&utm_medium=notification
<_whitenotifier-5> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/635237809?utm_source=github_status&utm_medium=notification
<_whitenotifier-5> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/635238260?utm_source=github_status&utm_medium=notification
rohitksingh has quit [Ping timeout: 260 seconds]
sb0 has quit [Quit: Leaving]
<_whitenotifier-5> [nmigen] Success. Absolute coverage decreased by -<.01% but relative coverage increased by +17.86% compared to 476ce15 - https://codecov.io/gh/m-labs/nmigen/compare/476ce15f041cbca08ba8bea10b05e75834c4074a...b057fb93ef7f48b24856c3c24ddf56c57be8543e
<_whitenotifier-5> [nmigen] Success. The Travis CI build passed - https://travis-ci.org/m-labs/nmigen/builds/635238260?utm_source=github_status&utm_medium=notification
<_whitenotifier-5> [nmigen] Success. Absolute coverage decreased by -<.01% but relative coverage increased by +17.86% compared to 8184efd - https://codecov.io/gh/m-labs/nmigen/compare/8184efd61215c575848fd51c60977f405a17ea36...b057fb93ef7f48b24856c3c24ddf56c57be8543e
mauz555 has joined #m-labs
<ZirconiumX> wq: https://gist.github.com/ZirconiumX/fd9826a9b452bd566b5ef096459888a8 <-- nMigen IL bug for $sshl
mauz555 has quit []
<_whitenotifier-5> [nmigen] ZirconiumX commented on issue #297: Output enable for Intel platform on multi I/O pins - https://git.io/JvvdZ
<ZirconiumX> Okay, so the bug is essentially nMigen not handling "x << -1" very well
<ZirconiumX> "x << (-1 & 63)" is more like the behaviour I wanted, and it works around the problem
X-Scale` has joined #m-labs
X-Scale has quit [Ping timeout: 268 seconds]
X-Scale` is now known as X-Scale
dlrobertson has quit [Quit: WeeChat 2.7]
proteus-guy has joined #m-labs
JK83 has joined #m-labs
<JK83> Hi, I just reported the same observation with issue #4 with SMA-DIO and updated on github.
<JK83> Please take a look and tell me if you have any idea.