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
stekern_ has joined #m-labs
stekern has quit [Read error: Connection reset by peer]
stekern has joined #m-labs
stekern_ has quit [Read error: Connection reset by peer]
forrestv has quit [Read error: Connection reset by peer]
<mithro> sb0: the CSR CSV format is used heavily by litescope - I've also got some very hacky tools which look at the csv file too
<mithro> Hrm sb0 isn't here...
<mithro> sb0: '::' isn't really a valid Python identifier, so its likely that litescope would have to translate it back into '_' or something...
<rjo> mithro: the idea would be to make it *not* a valid python identifier to be able to unambiguously distinguish a.b from a_b
<mithro> Yeah, that seems reasonable
<rjo> mithro: i guess that litescope could just `Mock()` that hierarchy very quickly and intuitively.
<mithro> rjo: yeah, at the moment it just provides a flat namespace
<mithro> At some point I need to talk to you all about device tree generation again..
sb0 has joined #m-labs
<rjo> mithro: by the way. we really should get a bit of documentation into migen/misoc. litex would profit equally. from what we have seen in the last weeks especially migen Platform/constraints/connector/build documentation appears to be high value.
<mithro> What has happened in the last few weeks?
<rjo> conferences, lots of meetings
<mithro> rjo: I was chatting with sb0 about being able to annotate platform stuff with human comments which can be exposed to the firmware. Things like silkscreen labels for LEDs and connectors.
<mithro> So in your firmware you can provide stuff like "led on <silkscreen label>" without having to have the firmware be specific to every board
<rjo> mithro: yes. i saw that. but that's there, right? just needs documentation on best practices and API.
<rjo> mithro: "usual" pcb tools give you a refdes for an led and a label. i.e. you will use the label in the platform def.
<rjo> LD8, P5V0_GOOD etc
<rjo> label=value for LEDs.
<mithro> Well, sb0 wasn't all that convinced of the idea - I was thinking that the PlatformInfo thingy was for that but he suggested that should be removed
<mithro> Sorry, I'm on my phone so might disappear at any moment
<rjo> i never noticed/needed PlatformInfo and was always happy with the 'platform signal name == pcb label/value' method.
<mithro> I have a tool which takes the Opsis KiCad schematic and generates a UCF file with comments
<mithro> rjo: I would like to improve connector stuff a bit - it would be nice if there was a little bit more formal structure...
<mithro> I was playing with something like that...
rohitksingh_work has joined #m-labs
wingdu has joined #m-labs
wingdu has quit [Client Quit]
sb0 has quit [Quit: Leaving]
wingdu has joined #m-labs
wingdu has quit [Client Quit]
<GitHub146> [artiq] mntng opened pull request #818: edit test_rpctool.py (master...master) https://github.com/m-labs/artiq/pull/818
<GitHub98> [artiq] mntng closed pull request #816: add test for artiq_rpctool (master...master) https://github.com/m-labs/artiq/pull/816
<whitequark> who is "mntng" and why do they not use a git client?
Ultrasauce has quit [Remote host closed the connection]
<GitHub84> [artiq] whitequark pushed 1 new commit to master: https://github.com/m-labs/artiq/commit/f5e1d9568d8bc9e5aaa1501de49844925bb8d4e1
<GitHub84> artiq/master f5e1d95 whitequark: firmware: update smoltcp.
<bb-m-labs> build #752 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/752
<bb-m-labs> build #552 of artiq-win64-test is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-win64-test/builds/552
<bb-m-labs> build #1651 of artiq is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/1651
karltheawesome has joined #m-labs
tangrs has quit [Ping timeout: 240 seconds]
tangrs has joined #m-labs
rohitksingh_work has quit [Read error: Connection reset by peer]
rohitksingh has joined #m-labs
<GitHub67> [artiq] jordens opened issue #819: devices: pass calibration parameters into drivers https://github.com/m-labs/artiq/issues/819
<GitHub29> [artiq] r-srinivas opened issue #820: Display working tree in file explorer https://github.com/m-labs/artiq/issues/820
mumptai has joined #m-labs
rohitksingh has quit [Quit: Leaving.]
<rjo> mithro: aren't you duplicateing stuff that should be in the schematic? or just comments?
<rjo> whitequark: mntng is sb0's intern from hkust
<rjo> sb0: he should introduce himself here!
<rjo> she!
<rjo> whitequark: another thing. could you leave one sayma connected (micro-usb to lab and power)?
<rjo> whitequark: i'll play with spi/jtag/openocd
mumptai has quit [Quit: Verlassend]
<whitequark> rjo: okay
sb0 has joined #m-labs
<sb0> whitequark, while you're there you can also prepare the zotino card (hook up power, and if you have better ones replace the hex standoffs and screws)
<sb0> there is a 12V power adapter on the board table that in theory is suitable AFAICT. but if you need it for sayma then sayma has priority.
<GitHub123> [artiq] sbourdeauducq commented on pull request #818 fdf7887: Also, 3249 is already used by another ARTIQ service. Better pick another port, e.g. 7777 as used in test_pc_rpc.py. https://github.com/m-labs/artiq/pull/818#discussion_r135154869
<GitHub1> [artiq] sbourdeauducq commented on pull request #818 fdf7887: This will fail if ``sys.executable`` contains a space character. Better pass the arguments with a list and use ``create_subprocess_exec`` as done elsewhere. https://github.com/m-labs/artiq/pull/818#discussion_r135154504
<GitHub165> [artiq] sbourdeauducq commented on pull request #818 fdf7887: ``None`` is the default. This line can be removed. https://github.com/m-labs/artiq/pull/818#discussion_r135154351
<GitHub47> [artiq] sbourdeauducq commented on pull request #818 fdf7887: Only one blank line is enough. https://github.com/m-labs/artiq/pull/818#discussion_r135155431
<GitHub44> [artiq] sbourdeauducq commented on pull request #818 fdf7887: Separate from system imports with a blank line. https://github.com/m-labs/artiq/pull/818#discussion_r135154150
<GitHub26> [artiq] sbourdeauducq commented on pull request #818 fdf7887: Underscore separation is the naming convention for functions, not classes. How about ``TestRPCTool``? https://github.com/m-labs/artiq/pull/818#discussion_r135155267
<GitHub131> [artiq] sbourdeauducq commented on pull request #818 fdf7887: You could just do ``proc = await asyncio.create_subprocess_exec(...) ... await proc.communicate()...`` https://github.com/m-labs/artiq/pull/818#discussion_r135155134