16:39 <hackerfoo> What's the right way to submit a PR for VTR? The octopus merge in master+wip causes the DCO check to fail because it doesn't have a sign-off.

17:01 <litghost> hackerfoo: Can you be more specific? Are you trying to make a PR for Symbiflow VTR, e.g. the generalized lookahead sampling?

17:01 <litghost> hackerfoo: Or are you trying to make a PR against upstream VTR?

17:34 <hackerfoo> litghost: I'm trying to get #289 ready (https://github.com/SymbiFlow/vtr-verilog-to-routing/pull/289), and I want it to run CI, but it falls on the DCO check.

17:34 <tpb> Title: Sample costs using one rr_node per segment type by HackerFoo · Pull Request #289 · SymbiFlow/vtr-verilog-to-routing · GitHub (at github.com)

17:34 <litghost> I've already provided you instructions in https://github.com/SymbiFlow/vtr-verilog-to-routing/pull/289#issuecomment-540614025 ?

17:34 <tpb> Title: Sample costs using one rr_node per segment type by HackerFoo · Pull Request #289 · SymbiFlow/vtr-verilog-to-routing · GitHub (at github.com)

17:47 <hackerfoo> litghost: Thanks

17:53 <mithro> Diagrams! - https://mithro-sphinxcontrib-verilog-diagrams.readthedocs.io/en/latest/

17:53 <tpb> Title: Sphinx Verilog Diagrams Sphinx Verilog documentation (at mithro-sphinxcontrib-verilog-diagrams.readthedocs.io)

21:58 <hackerfoo> How do I fix this? Error 1: /home/dusty/src/symbiflow-arch-defs-master/build/xc7/archs/artix7/devices/xc7a50t-basys3-roi-virt/arch.timing.xml:435 Unexpected attribute 'priority' found on node 'site'. Expected (possibly) one of: 'pb_type'.

21:59 <hackerfoo> Using symbiflow-arch-defs/master and https://github.com/SymbiFlow/vtr-verilog-to-routing/pull/298

21:59 <tpb> Title: New lookahead sampling method by HackerFoo · Pull Request #298 · SymbiFlow/vtr-verilog-to-routing · GitHub (at github.com)

21:59 <litghost> You are running an compatible version of VTR and symbiflow. How did you compile VTR?

21:59 <litghost> incompatible*

22:00 <hackerfoo> I compiled VTR

22:00 <hackerfoo> from my PR

22:00 <litghost> Ya, but with what?

22:00 <hackerfoo> make -j`nproc`

22:00 <litghost> Ya, you need all of the wip branches, not just that one

22:00 <hackerfoo> Okay. What's the right way to test my changes?

22:01 <hackerfoo> I have to merge with master+wip?

22:01 <litghost> Checkout master (not master+wip) and octopus merge all wip branches (including

22:01 <litghost> Your change modifies wip/connection_box..., so modify that wip branch to point to yours

22:03 <hackerfoo> Is there a document on any of this?

22:03 <litghost> acomodi and mithro came up with it, I believe they have a document?

22:03 <litghost> https://docs.google.com/document/d/1JVWkXxeSanAna8IRt0I49sG4aFS3Unlq693-HsQ0B0U/edit

22:03 <tpb> Title: SymbiFlow Process for Managing non-SymbiFlow Owned Projects - Google Docs (at docs.google.com)