Chick Markley

Results 147 comments of Chick Markley

Sample firrtl file ``` ;buildInfoPackage: chisel3, version: 3.5-SNAPSHOT, scalaVersion: 2.12.13, sbtVersion: 1.5.2 circuit Foo : module Baz : input clock : Clock input reset : AsyncReset output out : UInt...

Annotations example for above firrtl ``` [ { "class":"firrtl.stage.FirrtlFileAnnotation", "file":"test_run_dir/registers_with_presets_in_submodules_should_get_set_at_beginning/Foo.lo.fir" }, { "class":"firrtl.transforms.DedupedResult", "original":"~Foo|Baz", "duplicate":"~Foo|Foo/sub1:Bar/sub3:Baz", "index":0.0 }, { "class":"firrtl.transforms.DedupedResult", "original":"~Foo|Baz_1", "duplicate":"~Foo|Foo/sub1:Bar/sub4:Baz", "index":0.16666666666666666 }, { "class":"firrtl.transforms.DedupedResult", "original":"~Foo|Bar", "duplicate":"~Foo|Foo/sub1:Bar", "index":0.3333333333333333 }, {...

@jackkoenig It's a complete mystery, I'm trying to track this down now. xdg-open is mentioned only in one unused function in FirrtlDiagrammer object. I'm looking for clues still

I thought this would be fixed by PR #80 which forced the diagram open program to `""` for the tests. But CI still fails

Another approach is to build an executable jar. I just tried it on an ubuntu system. ``` > git clone https://github.com/freechipsproject/diagrammer.git > cd diagrammer > sbt assembly > history >...

Interesting. It's funny that the array method worked for you, when I did not see much difference. I started using an array method myself. Maybe I made the indexing method...

That's an awesome looking number. It does sound a little complicated. I've been making slow progress on getting the ast compiler code working. We'll need that to test more complicated...

Note: The chisel ShiftResetTester test, when run with version of chisel that tests using the interpreter, seems to be exhibit this sensitivity

Sorry, I've let this slide, can we talk about it at the hack session today

@shunshou I think the basic problem here is that it must be Reg of Vec instead of Vec of Reg. Issue #/chisel3/issues/352 shows an alternative way of representing this. We'd...