-
Notifications
You must be signed in to change notification settings - Fork 415
VTR flow library documentation is just a skeleton -- should fill in #2059
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
This issue has been inactive for a year and has been marked as stale. It will be closed in 15 days if it continues to be stale. If you believe this is still an issue, please add a comment. |
@jgoeders : we should close this one out. We still have just a skeleton of documentation for the VTR flow library. If you or a delegate can fill it out some that would be best. |
@WhiteNinjaZ Could you work on documenting this? |
Yea I can fill this in |
It looks like the documentation has been added already, at least when I build the docs locally, I get the generated docstrings for each of the functions. It looks like the web doc is still blank though. @vaughnbetz it is my understanding that the web docs are automatically updated with each PR? |
The CI automatically updates the docs. Not sure why they don't show up then ... I'm guessing there is something wrong with the doxyfile that says which ones to show on the web. See https://docs.verilogtorouting.org/en/latest/dev/code_documentation/ |
In case what I wrote is confusing: to make the web docs we need to do two things:
|
https://docs.verilogtorouting.org/en/latest/vtr/python_libs/vtr/ has the skeleton of documentation for this library, but only the headings are there.
Proposed Behaviour
Complete the documentation
Context
Our flow scripts are pretty complex (and useful) so documenting them is helpful. We do have pretty reasonable documentation on the overall scripts, so if you conclude no documentation is needed on the library you could instead delete this documentation tab. But I suspect documentation here will be useful.
The text was updated successfully, but these errors were encountered: