w0lek

Results 13 comments of w0lek

@volodymyrkochyn @ravikiranchollangi For Interactive Path Analysis tool we have following code, the place when we create NCriticalPathWidget and insert it as the tab. ``` NCriticalPathWidget* viewWidget = new NCriticalPathWidget(compiler); viewWidget->setProperty("deleteOnCloseTab",...

Unresolved tasks from https://github.com/verilog-to-routing/vtr-verilog-to-routing/pull/2483 1. Add end user documentation (document the command line option, and this feature in the graphics section (probably a new tab). 2. We should have Doxygen...

@soheilshahrouz Up to now i have two failures in CI: **1) Python Lint** and **2) Check Compilation Warnings**. But i am not sure if failure caused by mine changes or...

@soheilshahrouz i moved out PR from a draft. The user end docu will be made in separate PR. I fixed CI warning tests and Python Lint. However the Python Lint...

> Thank you for the PR! I also noticed that there are some warnings introduced by this PR. I have been working on trying to remove all of the warnings....

TODO: use snakecases for variable names, function names and method names. "{" must be on the same line as ")" in function definition.

> TODO: use snakecases for variable names, function names and method names. "{" must be on the same line as ")" in function definition. this is done

@AlexandreSinger to fix warning compilation in sockpp thirdparty library, i created fork with custom branch where the warnings were fixed, and set using this fork instead of upstream in vpr...

@soheilshahrouz thank you for reviewing PR. i have finished all review tasks, until there will no be new one found. In short sumup of some relevant changes: - all new...

@AlexandreSinger thank you for detailed explanation, i have added test case when GRAPHICS=on but SERVER=off https://github.com/verilog-to-routing/vtr-verilog-to-routing/pull/2526/commits/761c1be7cfe2d9518e47c9b060c0c6e9737668ed Also FYI: 1) the end user doc PR is ready to be created, as...