vtr-verilog-to-routing icon indicating copy to clipboard operation
vtr-verilog-to-routing copied to clipboard

Ability to act as a server using a socket communication link, handle critical path list requests, and highlight certain critical path requests

Open w0lek opened this issue 6 months ago • 1 comments

vpr_it1_pullrequest.pdf

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

  • [ ] Bug fix (change which fixes an issue)
  • [ ] New feature (change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [ ] My change requires a change to the documentation
  • [ ] I have updated the documentation accordingly
  • [ ] I have added tests to cover my changes
  • [ ] All new and existing tests passed

w0lek avatar Feb 01 '24 17:02 w0lek

Thanks for developing what looks like a great new feature.

  1. Please remove the OpenFPGA tilable rr-graph files from the PR; we should focus on the GUI here.
  2. Add end user documentation (document the command line option, and this feature in the graphics section (probably a new tab).
  3. We should have Doxygen comments on the server etc. code (which I think you already have) and add the resulting hyperlinked documentation to read the docs, probably under VPR API. There is guidance on how to do that in the VTR Developer Guide (it's a short amount of code to write once you have the Doxygen comments, and the Guide tells you where to write it).

vaughnbetz avatar Feb 01 '24 18:02 vaughnbetz

Superseded by #2526.

soheilshahrouz avatar Apr 20 '24 17:04 soheilshahrouz