| *** tpb has joined #vtr-dev | 00:00 | |
| mithro | kem_: https://github.com/kmurray/libblifparse/pull/2 <- Code formatting starting with the deps repositories that you own | 16:16 |
|---|---|---|
| tpb | Title: Add format target for source code formatting by kgugala · Pull Request #2 · kmurray/libblifparse · GitHub (at github.com) | 16:16 |
| litghost | kem_: I think I have an example graph where the router fails to find a valid path because it doesn't add all valid fan outs to the heap. It sounds vaguely like what https://github.com/verilog-to-routing/vtr-verilog-to-routing/commit/46661d08602a71ed641a91e67fa2833cc1aefb62#diff-4ac57f609972e5b875fd99f90d6f518e was supposed to addressed, but it is occurring on something close to master | 22:04 |
| tpb | Title: vpr: Improve robustness of high-fanout routing on some architectures · verilog-to-routing/vtr-verilog-to-routing@46661d0 · GitHub (at github.com) | 22:04 |
| litghost | kem_: Might be a poor interaction between non-configurable channel connections? | 22:05 |
| litghost | kem_: I think the bug is here https://github.com/verilog-to-routing/vtr-verilog-to-routing/blob/master/vpr/src/route/route_timing.cpp#L1855 | 22:08 |
| tpb | Title: vtr-verilog-to-routing/route_timing.cpp at master · verilog-to-routing/vtr-verilog-to-routing · GitHub (at github.com) | 22:08 |
| litghost | kem_: Basically during non-configurable edge expansion, the heap gets a downstream node from the source, but in a case where there is a high-fanout non-configurable net, it is possible that the wrong node is added to the heap | 22:09 |
| litghost | kem_: Until a valid route is found, all edges need to be expanded, yes? But that doesn't appear to be happening | 22:09 |
| litghost | kem_: Non-configurable seems to be a red herring | 22:29 |
| litghost | kem_: Ah, found the issue, https://github.com/verilog-to-routing/vtr-verilog-to-routing/blob/master/vpr/src/route/route_timing.cpp#L1751 is filtering the route | 22:41 |
| tpb | Title: vtr-verilog-to-routing/route_timing.cpp at master · verilog-to-routing/vtr-verilog-to-routing · GitHub (at github.com) | 22:41 |
| mithro | litghost: https://vtr-verilog-to-routing.readthedocs.io/en/latest/vpr/command_line_usage.html#cmdoption-vpr-const-gen-inference | 23:11 |
| tpb | Title: Command-line Options Verilog-to-Routing 8.0.0-dev documentation (at vtr-verilog-to-routing.readthedocs.io) | 23:11 |
| mithro | litghost: btw do you have VTR_ENABLE_DEBUG_LOGGING on? | 23:20 |
| litghost | mithro: Ya | 23:20 |
Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!