sb0 changed the topic of #m-labs to: ARTIQ, Migen, MiSoC, Mixxeo & other M-Labs projects :: fka #milkymist :: Logs http://irclog.whitequark.org/m-labs
[X-Scale] has joined #m-labs
X-Scale has quit [Ping timeout: 245 seconds]
[X-Scale] is now known as X-Scale
ardavast has quit [Read error: Connection reset by peer]
ardavast has joined #m-labs
<sb0> whitequark, no, just reload the bitstream
<whitequark> ok
ardavast has quit [Read error: Connection reset by peer]
ardavast has joined #m-labs
futarisIRCcloud has joined #m-labs
<sb0> is it OK to connect two Kasli EEM connectors together with an IDC cable? I can't see why not, but with 12V around I'd rather double-check than do something dumb
rohitksingh_work has joined #m-labs
ardavast has quit [Read error: Connection reset by peer]
ardavast has joined #m-labs
X-Scale has quit [Ping timeout: 248 seconds]
<GitHub124> [smoltcp] hjr3 opened pull request #218: Add IPv6 Extension Routing Header (master...ipv6routing) https://github.com/m-labs/smoltcp/pull/218
<GitHub183> [smoltcp] hjr3 commented on issue #218: @dlrobertson PR is up for IPv6 routing. https://github.com/m-labs/smoltcp/pull/218#issuecomment-390865178
futarisIRCcloud has quit [Quit: Connection closed for inactivity]
ardavast has quit [Read error: Connection reset by peer]
ardavast has joined #m-labs
<rjo> cr1901_modern: yes. round to kHz for example.
<rjo> whitequark: you need to release the board lock when you are done with it
ardavast_ has joined #m-labs
ardavast has quit [Read error: Connection reset by peer]
Gurty has quit [Ping timeout: 265 seconds]
ardavast_ has quit [Read error: Connection reset by peer]
ardavast has joined #m-labs
Gurty has joined #m-labs
Gurty has joined #m-labs
Gurty has quit [Changing host]
ardavast has quit [Read error: Connection reset by peer]
ardavast has joined #m-labs
rohitksingh_work has quit [Read error: Connection reset by peer]
<rjo> whitequark: i'll be at ptb tomorrow. how are the chances of getting the critical issues fixed in master?
<GitHub148> [smoltcp] dlrobertson commented on pull request #218 520a802: https://tools.ietf.org/html/rfc4727 https://github.com/m-labs/smoltcp/pull/218#discussion_r189896386
<GitHub142> [smoltcp] dlrobertson commented on pull request #218 520a802: Could type specific packet methods be in separate documented `impl` blocks. See `ndisc.rs`. https://github.com/m-labs/smoltcp/pull/218#discussion_r189899082
<GitHub58> [smoltcp] dlrobertson commented on pull request #218 520a802: https://tools.ietf.org/html/rfc6554 https://github.com/m-labs/smoltcp/pull/218#discussion_r189896240
<GitHub150> [smoltcp] dlrobertson commented on pull request #218 520a802: https://tools.ietf.org/html/rfc4727 https://github.com/m-labs/smoltcp/pull/218#discussion_r189896415
<GitHub188> [smoltcp] dlrobertson commented on pull request #218 520a802: Use the ietf. The iana links just 302 to the ietf. https://tools.ietf.org/html/rfc5095 https://github.com/m-labs/smoltcp/pull/218#discussion_r189896044
<GitHub198> [smoltcp] dlrobertson commented on pull request #218 520a802: Is this really used? Type 2 uses `home_address` and RPL uses `address`. https://github.com/m-labs/smoltcp/pull/218#discussion_r189899778
<GitHub16> [smoltcp] hjr3 commented on pull request #218 24d307a: fixed https://github.com/m-labs/smoltcp/pull/218#discussion_r189907020
<GitHub45> [smoltcp] hjr3 commented on pull request #218 24d307a: fixed https://github.com/m-labs/smoltcp/pull/218#discussion_r189907063
<GitHub66> [smoltcp] hjr3 commented on pull request #218 24d307a: fixed https://github.com/m-labs/smoltcp/pull/218#discussion_r189907083
<GitHub106> [smoltcp] hjr3 commented on pull request #218 24d307a: fixed https://github.com/m-labs/smoltcp/pull/218#discussion_r189907105
<GitHub117> [smoltcp] hjr3 commented on pull request #218 24d307a: I had this but thought we were moving away from it with only panic comments. Done. https://github.com/m-labs/smoltcp/pull/218#discussion_r189907399
<GitHub184> [smoltcp] hjr3 commented on pull request #218 24d307a: It was used in a test. I removed `data` and `data_mut` and modified the test. https://github.com/m-labs/smoltcp/pull/218#discussion_r189907513
X-Scale has joined #m-labs
<GitHub141> [smoltcp] hjr3 commented on pull request #218 24d307a: Should I panic on a value of `0` then? That is a valid routing type, though not one this code explicitly supports. https://github.com/m-labs/smoltcp/pull/218#discussion_r189909970
rohitksingh has joined #m-labs
ardavast has quit [Read error: Connection reset by peer]
ardavast has joined #m-labs
<GitHub-m-labs> [artiq] jordens opened issue #1009: len(Scannable) support broken https://github.com/m-labs/artiq/issues/1009
<whitequark> rjo: yes. sorry about that.
<whitequark> I'm fixing them right now.
<bb-m-labs> build #2365 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2365 blamelist: Robert Jordens <rj@m-labs.hk>
<rjo> whitequark: ack
<GitHub-m-labs> [artiq] jordens commented on issue #1009: Using `self.scan.sequence` seems to be a workaround. https://github.com/m-labs/artiq/issues/1009#issuecomment-391037276
<GitHub175> [smoltcp] whitequark commented on pull request #218 24d307a: Yes, since you do not actually clear the reserved fields, you should panic. https://github.com/m-labs/smoltcp/pull/218#discussion_r189958958
<GitHub1> [smoltcp] whitequark commented on pull request #218 24d307a: Are these `pub use`s really necessary? They aren't exported in `mod wire`, so the `pub` is useless. https://github.com/m-labs/smoltcp/pull/218#discussion_r189958547
<GitHub33> [smoltcp] hjr3 commented on pull request #218 5298144: I forgot why we were doing this. It is in other extension headers as well: https://github.com/m-labs/smoltcp/blob/master/src/wire/ipv6fragment.rs#L6 and https://github.com/m-labs/smoltcp/blob/master/src/wire/ipv6hopbyhop.rs#L4... https://github.com/m-labs/smoltcp/pull/218#discussion_r189963749
<GitHub16> [smoltcp] hjr3 commented on pull request #218 5298144: Makes sense. Done. https://github.com/m-labs/smoltcp/pull/218#discussion_r189963806
<GitHub104> [smoltcp] dlrobertson commented on pull request #218 5298144: This repr doesn't need `length` or `segments_left` they will always be 2 and 1 respectively. It is an invalid packet if they are not. https://github.com/m-labs/smoltcp/pull/218#discussion_r189965200
ardavast has quit [Remote host closed the connection]
ardavast has joined #m-labs
<GitHub155> [smoltcp] hjr3 commented on pull request #218 5298144: I went back and forth on this. I decided to leave them explicit as I imagined that downstream code would not want these values implicit. If I remove those fields, should I provide methods on the Repr that return the number of length and number of segments left?... https://github.com/m-labs/smoltcp/pull/218#discussion_r189968694
<GitHub13> [smoltcp] dlrobertson commented on pull request #218 5298144: > I decided to leave them explicit as I imagined that downstream code would not want these values implicit.... https://github.com/m-labs/smoltcp/pull/218#discussion_r189971416
<GitHub188> [smoltcp] dlrobertson commented on pull request #218 5298144: The other instances of `pub use super::IpProtocol as Protocol` also do not need to be `pub`. https://github.com/m-labs/smoltcp/pull/218#discussion_r189973118
<GitHub-m-labs> [artiq] jordens pushed 2 new commits to master: https://github.com/m-labs/artiq/compare/4e863b32a170...fcd12e34728a
<GitHub-m-labs> artiq/master fcd12e3 Robert Jordens: suservo: work around #1007
<GitHub-m-labs> artiq/master 9a86dfe Robert Jordens: units: add mHz
<GitHub-m-labs> [artiq] whitequark pushed 1 new commit to master: https://github.com/m-labs/artiq/commit/ce3c8fcd3e7b3d604d4bc56ce882f557a39917dd
<GitHub-m-labs> artiq/master ce3c8fc whitequark: compiler: add source location to SSA arguments....
<GitHub-m-labs> [artiq] whitequark commented on issue #979: @jordens I've fixed #986. In case you manage to reproduce this, you can gather and view a profile with:... https://github.com/m-labs/artiq/issues/979#issuecomment-386769720
<GitHub-m-labs> [artiq] whitequark commented on issue #1009: No, this isn't supposed to be supported--`len` is currently only defined for builtin types, and `__len__` is ignored. https://github.com/m-labs/artiq/issues/1009#issuecomment-391090977
<GitHub-m-labs> [artiq] whitequark commented on issue #1009: No, this isn't supposed to be supported--`len` is currently only defined for builtin types, and `__len__` is ignored. Same for `__iter__`. https://github.com/m-labs/artiq/issues/1009#issuecomment-391090977
<GitHub-m-labs> [artiq] whitequark closed issue #1009: len(Scannable) support broken https://github.com/m-labs/artiq/issues/1009
<bb-m-labs> build #1544 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/1544
<bb-m-labs> build #2366 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2366 blamelist: Robert Jordens <jordens@gmail.com>
ardavast has quit [Remote host closed the connection]
ardavast has joined #m-labs
rohitksingh has quit [Quit: Leaving.]
<GitHub84> [dslite2svd] thejpster tagged v0.7.0 at master: https://github.com/m-labs/dslite2svd/commits/v0.7.0
<GitHub-m-labs> [artiq] whitequark pushed 1 new commit to master: https://github.com/m-labs/artiq/commit/0c6f6180560c9e51e1098107afe21c7070e8365d
<GitHub-m-labs> artiq/master 0c6f618 whitequark: manual: update to reflect the new artiq_coremgmt tool.
<GitHub-m-labs> [artiq] whitequark closed issue #1008: manual: update/remove/fix references to artiq_corelog etc https://github.com/m-labs/artiq/issues/1008
<bb-m-labs> build #1545 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/1545
<bb-m-labs> build #2367 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2367 blamelist: whitequark <whitequark@whitequark.org>
<bb-m-labs> build #1546 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/1546
<bb-m-labs> build #2368 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2368 blamelist: whitequark <whitequark@whitequark.org>
ardavast has quit [Read error: Connection reset by peer]
ardavast has joined #m-labs
<GitHub-m-labs> [artiq] hartytp commented on issue #788: @jbqubit I definitely agree with you that this all needs more documentation. However, I don't think that much of it is SU-servo specific. e.g. Kasli, Urukul and Sampler all need proper docs (as do the other EEMs), explaining the connections they need in a way that physicists understand. We also need some higher-level docs explaining how the Kasli/EEM ecosystem works and
<GitHub-m-labs> [artiq] hartytp commented on issue #788: Oh, and I'd generally tend to run Urkul's system clock at 1GHz. But, IIRC, you can use more or less any Urukul clock frequency you want with the servo. Essentially, the servo just takes control of the SPI port on Urkul, but leaves everything else alone. The SPI port is independent of the clock frequency IIRC (but, in case of doubt/questions, refer to the AD9910 data shee
<whitequark> rjo: nope, can't fix #1007 until tomorrow
<whitequark> the transformation I need really wants to be an LLVM pass, but we don't have any infrastructure for ARTIQ-specific LLVM passes
<whitequark> actually, why on earth is test_clock_generator_loopback using FP at all?
<whitequark> ah right, frequency is in FP. but 2**acc_width could be 1<<acc_width instead...
<whitequark> it doesn't help very much though
<GitHub-m-labs> [artiq] jordens commented on issue #1008: d446a3293ec2869aef4b2ec846b553975a38a01d also removes `aqctl_corelog`. That's a controller which is used throughout the examples. That was probably unintentional and needs to be reverted or fixed.... https://github.com/m-labs/artiq/issues/1008#issuecomment-391147028
<GitHub-m-labs> [artiq] jordens commented on issue #1009: If `__iter__` is not supported on the core device, it should not be `portable`. https://github.com/m-labs/artiq/issues/1009#issuecomment-391147699
<14WAABF4S> [artiq] whitequark pushed 2 new commits to master: https://github.com/m-labs/artiq/compare/0c6f6180560c...dfb4a437f5ff
<14WAABF4S> artiq/master dfb4a43 whitequark: manual: mention TRACE log level.
<14WAABF4S> artiq/master b55ce66 whitequark: Reinstate accidentally removed aqctl_corelog....
<94KAABE13> [artiq] whitequark closed issue #1008: manual: update/remove/fix references to artiq_corelog etc https://github.com/m-labs/artiq/issues/1008
<GitHub-m-labs> [artiq] whitequark commented on issue #1009: That predates the new compiler. https://github.com/m-labs/artiq/issues/1009#issuecomment-391147937
<GitHub-m-labs> [artiq] whitequark pushed 1 new commit to master: https://github.com/m-labs/artiq/commit/7aacc04f0ca7f5d7eaf0a2ad569599a76a7abd2d
<GitHub-m-labs> artiq/master 7aacc04 whitequark: language: scan functions are not supported on core device....
<GitHub-m-labs> [artiq] jordens commented on issue #1009: ACK. Thanks. https://github.com/m-labs/artiq/issues/1009#issuecomment-391148271
<rjo> whitequark: ok. thanks for letting me know. i'll have to work around it.
<GitHub195> [smoltcp] dlrobertson commented on issue #218: @m-labs-homu r=whitequark https://github.com/m-labs/smoltcp/pull/218#issuecomment-391152456
<GitHub1> [smoltcp] m-labs-homu commented on issue #218: :pushpin: Commit 5298144 has been approved by `whitequark`
<GitHub198> [smoltcp] m-labs-homu pushed 3 new commits to auto: https://github.com/m-labs/smoltcp/compare/602724b2858e...9c1900ff3f66
<GitHub198> smoltcp/auto 9c1900f Herman J. Radtke III: CR fixes deux...
<GitHub198> smoltcp/auto b789e6e Herman J. Radtke III: CR fixes...
<GitHub198> smoltcp/auto a0009a0 Herman J. Radtke III: Add IPv6 Extension Routing Header...
<GitHub122> [smoltcp] m-labs-homu commented on issue #218: :hourglass: Testing commit 52981446b91011d2614ce433d8d7f6d864f52d59 with merge 9c1900ff3f66196f659cbfc79213dc944196db9d... https://github.com/m-labs/smoltcp/pull/218#issuecomment-391152554
<GitHub94> [smoltcp] hjr3 closed issue #160: Design: Implement IPv6 Extension Routing Header https://github.com/m-labs/smoltcp/issues/160
<GitHub198> [smoltcp] hjr3 commented on issue #160: Closed by a0009a0b614b71faffb3f0bd3490357e731baaa9 https://github.com/m-labs/smoltcp/issues/160#issuecomment-391153884
<travis-ci> m-labs/smoltcp#966 (auto - 9c1900f : Herman J. Radtke III): The build passed.
<GitHub71> [smoltcp] m-labs-homu merged auto into master: https://github.com/m-labs/smoltcp/compare/602724b2858e...9c1900ff3f66
<GitHub105> [smoltcp] m-labs-homu commented on issue #218: :sunny: Test successful - [status-travis](https://travis-ci.org/m-labs/smoltcp/builds/382389426?utm_source=github_status&utm_medium=notification)
<GitHub85> [smoltcp] m-labs-homu closed pull request #218: Add IPv6 Extension Routing Header (master...ipv6routing) https://github.com/m-labs/smoltcp/pull/218
<travis-ci> m-labs/smoltcp#967 (master - 9c1900f : Herman J. Radtke III): The build passed.
<GitHub-m-labs> [artiq] jordens pushed 1 new commit to master: https://github.com/m-labs/artiq/commit/9715f774ac639b4cad8e62adc781800ba65c1dde
<GitHub-m-labs> artiq/master 9715f77 Robert Jordens: artiq_influxdb: controller (add ping())
<bb-m-labs> build #1547 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/1547
<bb-m-labs> build #2369 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2369 blamelist: whitequark <whitequark@whitequark.org>
ardavast has quit [Read error: Connection reset by peer]
ardavast_ has joined #m-labs
<bb-m-labs> build #1548 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/1548
<bb-m-labs> build #2370 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2370 blamelist: whitequark <whitequark@whitequark.org>, Robert Jordens <jordens@gmail.com>