sb0_ changed the topic of #m-labs to: https://m-labs.hk :: Logs http://irclog.whitequark.org/m-labs
d_n|a has joined #m-labs
<d_n|a> bb-m-labs: force build --branch=pull/1188/merge artiq
<bb-m-labs> build forced [ETA 1h00m54s]
<bb-m-labs> I'll give a shout when the build finishes
zng has quit [Ping timeout: 268 seconds]
zng has joined #m-labs
<bb-m-labs> build #2123 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2123
<bb-m-labs> build #2124 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2124
<bb-m-labs> build #2752 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2752
<GitHub-m-labs> [artiq] klickverbot pushed 1 new commit to master: https://github.com/m-labs/artiq/commit/d4c393b2a8b1b30debab5b02b60cb3f5a6787219
<GitHub-m-labs> artiq/master d4c393b David Nadlinger: firmware/ksupport: Update `cfg(not(has_rtio))` stub signatures...
<bb-m-labs> build #2125 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2125
<bb-m-labs> build #2126 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2126
<bb-m-labs> build #966 of artiq-win64-test is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-win64-test/builds/966
<bb-m-labs> build #2753 of artiq is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2753
<mithro> I can't figure out what is going on here -- With a CSRStorage with write_from_dev=True, asserting csr.we should cause dat_w to be copied into storage_full...
<mithro> Oh...
<mithro> # CSRStorage(write_from_dev=True) broken in sim
rohitksingh_work has joined #m-labs
felix_ has quit [Ping timeout: 246 seconds]
felix_ has joined #m-labs
<mithro> sb0: Why would write_from_dev be broken in simulation?
<whitequark> mithro: no particular opinion
<GitHub> [pythonparser] whitequark pushed 1 new commit to master: https://github.com/m-labs/pythonparser/commit/c31a65febd7800e8f9e4f5347d30d9beec786a7c
<GitHub> pythonparser/master c31a65f Alan Justino: parser.Parser explicitly inheriting from object...
<GitHub> [pythonparser] whitequark pushed 1 new commit to master: https://github.com/m-labs/pythonparser/commit/6dd870e7b004f23b96f43fc22119752a15c96831
<GitHub> pythonparser/master 6dd870e Alan Justino: Accepts empty print_stmt with destination fp...
<mithro> whitequark: Any idea why the write_from_dev be broken in simulation
<whitequark> I don't know
<whitequark> I don't write a lot of code that uses CSRs
rohitksingh has joined #m-labs
rohitksingh has quit [Read error: Connection reset by peer]
rohitksingh has joined #m-labs
_whitelogger has joined #m-labs
rohitksingh has quit [Ping timeout: 268 seconds]
<mithro> whitequark: It seems the issue is that the CSRStorage has a do_finalize method which isn't called because the CSR isn't a submodule...
futarisIRCcloud has joined #m-labs
siruf_ has joined #m-labs
siruf has quit [Ping timeout: 250 seconds]
siruf_ is now known as siruf
<GitHub-m-labs> [artiq] jordens pushed 1 new commit to release-4: https://github.com/m-labs/artiq/commit/a7d7e91188846669939c634818e8d6ff5056fff8
<GitHub-m-labs> artiq/release-4 a7d7e91 David Nadlinger: firmware/ksupport: Update `cfg(not(has_rtio))` stub signatures...
<GitHub-m-labs> [artiq] sbourdeauducq commented on commit a7d7e91: @klickverbot Thanks for fixing this. ``not(has_rtio)`` was something for Sayma trench warfare and it can be removed, maybe after Sayma v2. https://github.com/m-labs/artiq/commit/a7d7e91188846669939c634818e8d6ff5056fff8#commitcomment-31632008
<GitHub-m-labs> [artiq] sbourdeauducq pushed 1 new commit to release-4: https://github.com/m-labs/artiq/commit/e4ddfb303cd14be1275d1b0381696fccf4f7b426
<GitHub-m-labs> artiq/release-4 e4ddfb3 Sebastien Bourdeauducq: Revert "firmware/ksupport: Update `cfg(not(has_rtio))` stub signatures"...
<bb-m-labs> build #2127 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2127
rohitksingh_work has quit [Read error: Connection reset by peer]
rohitksingh_work has joined #m-labs
<bb-m-labs> build #2128 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2128
<bb-m-labs> build #2754 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2754 blamelist: David Nadlinger <code@klickverbot.at>
futarisIRCcloud has quit [Quit: Connection closed for inactivity]
<bb-m-labs> build #2129 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2129
<bb-m-labs> build #2130 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2130
<bb-m-labs> build #967 of artiq-win64-test is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-win64-test/builds/967
<bb-m-labs> build #2755 of artiq is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2755
<GitHub-m-labs> [artiq] jordens pushed 2 new commits to master: https://github.com/m-labs/artiq/compare/d4c393b2a8b1...6df4ae934f07
<GitHub-m-labs> artiq/master efd400b Robert Jördens: ad9910: style [nfc]...
<GitHub-m-labs> artiq/master 6df4ae9 Robert Jördens: eem: name the servo submodule...
<GitHub-m-labs> [artiq] jordens pushed 1 new commit to release-4: https://github.com/m-labs/artiq/commit/b1ea02ddf0c0feed2a99d0def10e513b21d70799
<GitHub-m-labs> artiq/release-4 b1ea02d Robert Jördens: eem: name the servo submodule...
rohitksingh_work has quit [Read error: Connection reset by peer]
<GitHub-m-labs> [artiq] jordens opened issue #1207: unification prevents simultaneous use of coredevices in different configurations https://github.com/m-labs/artiq/issues/1207
<GitHub-m-labs> [artiq] jordens commented on issue #1207: This works:... https://github.com/m-labs/artiq/issues/1207#issuecomment-446163302
<GitHub-m-labs> [artiq] jordens commented on issue #1207: @whitequark is it by accident that this works and does the type annotation work around it?... https://github.com/m-labs/artiq/issues/1207#issuecomment-446164001
<bb-m-labs> build #2131 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2131
<bb-m-labs> build #2132 of artiq-board is complete: Failure [failed conda_build] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2132 blamelist: Robert J?rdens <rj@quartiq.de>
<bb-m-labs> build #2756 of artiq is complete: Failure [failed] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2756 blamelist: Robert J?rdens <rj@quartiq.de>
<GitHub-m-labs> [artiq] jordens reopened issue #1207: unification prevents use of devices in different configurations https://github.com/m-labs/artiq/issues/1207
<bb-m-labs> build #2133 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2133
<bb-m-labs> build #2134 of artiq-board is complete: Failure [failed conda_build] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2134 blamelist: Robert J?rdens <rj@quartiq.de>
<bb-m-labs> build #2135 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2135
<bb-m-labs> build #2757 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2757 blamelist: Robert J?rdens <rj@quartiq.de>
<rjo> sb0: fyi. i'm going to commit a bunch of ad9910-ram related stuff today.
key2 has joined #m-labs
<GitHub-m-labs> [artiq] jordens created ad9910-ram (+1 new commit): https://github.com/m-labs/artiq/commit/79eadb9465f6
<GitHub-m-labs> artiq/ad9910-ram 79eadb9 Robert Jördens: ad9910: add RAM mode methods...
<GitHub-m-labs> [artiq] jordens opened pull request #1208: ad9910: add RAM mode methods (master...ad9910-ram) https://github.com/m-labs/artiq/pull/1208
zng has quit [Quit: ZNC 1.8.x-nightly-20181129-f3eca21b - https://znc.in]
zng has joined #m-labs
zng has quit [Client Quit]
zng has joined #m-labs
<GitHub-m-labs> [artiq] whitequark commented on issue #1207: @jordens RPCs that return non-None values require a type annotation, by design. https://github.com/m-labs/artiq/issues/1207#issuecomment-446302228
<GitHub-m-labs> [artiq] jordens commented on issue #1207: Oh. Yeah. That was meant to be a kernel. https://github.com/m-labs/artiq/issues/1207#issuecomment-446332568
<mithro> sb0 / whitequark: I could use a review of https://github.com/enjoy-digital/litex/pull/131/files -- I'm not sure what I'm doing...
<GitHub-m-labs> [artiq] klickverbot opened pull request #1209: coredevice, firmware: Add rtio_input_timestamped_data (master...input-timestamped-data) https://github.com/m-labs/artiq/pull/1209
<d_n|a> bb-m-labs: force build --branch=pull/1209/merge artiq
<bb-m-labs> build forced [ETA 1h00m17s]
<bb-m-labs> I'll give a shout when the build finishes
<GitHub-m-labs> [artiq] whitequark commented on issue #1207: @jordens Can you provide an MCVE? I've read the examples above a few times and I still don't understand the issue. https://github.com/m-labs/artiq/issues/1207#issuecomment-446364961
<bb-m-labs> build #2758 of artiq is complete: Failure [failed python_unittest] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2758
<GitHub-m-labs> [artiq] sbourdeauducq commented on pull request #1185 01e4d70: That's obvious from the function name and its code. https://github.com/m-labs/artiq/pull/1185#discussion_r240797935
<GitHub-m-labs> [artiq] sbourdeauducq commented on pull request #1185 01e4d70: That's redundant with the module description. https://github.com/m-labs/artiq/pull/1185#discussion_r240798112
<GitHub-m-labs> [artiq] sbourdeauducq commented on pull request #1185 01e4d70: Wrong exception message. The user is never supposed to see this; if this code is reached it is a bug in the program and using ``-h`` is not going to hep. https://github.com/m-labs/artiq/pull/1185#discussion_r240798448
<GitHub-m-labs> [artiq] sbourdeauducq commented on pull request #1185 01e4d70: Wrong exception message. The user is never supposed to see this; if this code is reached it is a bug in the program and using ``-h`` is not going to help. https://github.com/m-labs/artiq/pull/1185#discussion_r240798448
<GitHub-m-labs> [artiq] sbourdeauducq commented on pull request #1185 01e4d70: Obvious. https://github.com/m-labs/artiq/pull/1185#discussion_r240798943
lkcl has quit [Ping timeout: 246 seconds]
<GitHub-m-labs> [artiq] sbourdeauducq commented on pull request #1185 01e4d70: OK https://github.com/m-labs/artiq/pull/1185#discussion_r240799051
<GitHub-m-labs> [artiq] sbourdeauducq commented on pull request #1185 01e4d70: "datasets" to use consistent terminology. https://github.com/m-labs/artiq/pull/1185#discussion_r240799320
<GitHub-m-labs> [artiq] sbourdeauducq commented on pull request #1209 ed3975e: Nitpicking but IIRC you can put ``#[repr(C)]`` on the function and return ``(u64, i32)``. This way it's a tuple in both Rust and Python; and not a structure in Rust and a tuple in Python. https://github.com/m-labs/artiq/pull/1209#discussion_r240800060
<GitHub-m-labs> [artiq] sbourdeauducq commented on pull request #1209 ed3975e: Nitpicking but IIRC you can put ``#[repr(C)]`` on the function and return ``(i64, i32)``. This way it's a tuple in both Rust and Python; and not a structure in Rust and a tuple in Python. https://github.com/m-labs/artiq/pull/1209#discussion_r240800060
<GitHub-m-labs> [artiq] sbourdeauducq commented on pull request #1209 ed3975e: Should be ``i64`` for ARTIQ-Python consistency. https://github.com/m-labs/artiq/pull/1209#discussion_r240800329
lkcl has joined #m-labs
<GitHub-m-labs> [artiq] jordens commented on issue #1207: This ~works~ doesn't work:... https://github.com/m-labs/artiq/issues/1207#issuecomment-446163302
<GitHub-m-labs> [artiq] jordens commented on issue #1207: ~@whitequark is it by accident that this works and does the type annotation work around it?... https://github.com/m-labs/artiq/issues/1207#issuecomment-446164001
<GitHub-m-labs> [artiq] jordens commented on issue #1207: @whitequark The MCVE is the first snippet posted. It's complete, verifiable and the most minimal I could come up with. Forget about the second snippet.... https://github.com/m-labs/artiq/issues/1207#issuecomment-446375417
<GitHub-m-labs> [artiq] klickverbot commented on pull request #1209 ed3975e: Ah, yes – that's much cleaner. https://github.com/m-labs/artiq/pull/1209#discussion_r240804231
<GitHub-m-labs> [artiq] klickverbot commented on pull request #1209 ed3975e: I just went with the `rtio_input_timestamp` return type, but will change. https://github.com/m-labs/artiq/pull/1209#discussion_r240804486
mumptai has joined #m-labs
<GitHub-m-labs> [artiq] qknight opened pull request #1210: Rust 1.28 (rust-1.28...rust-1.28) https://github.com/m-labs/artiq/pull/1210
mumptai has quit [Remote host closed the connection]