skywater-pdk icon indicating copy to clipboard operation
skywater-pdk copied to clipboard

docs: Add generator for cell index

Open PiotrZierhoffer opened this issue 5 years ago • 18 comments

This script creates list of cells in each library.

Addressing #86 .

PiotrZierhoffer avatar Aug 25 '20 11:08 PiotrZierhoffer

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Aug 25 '20 11:08 google-cla[bot]

Should I commit changed README.rst files in libraries? I added this line to each

.. include:: cell-list.rst

PiotrZierhoffer avatar Aug 26 '20 09:08 PiotrZierhoffer

Another update @mithro.

I tend to keep configuration like the rst templates etc at top level, as this is the first thing a user will want to change when altering the resulting rst, but it can be adjusted as you prefer.

How to proceed with changes to README.rst?

PiotrZierhoffer avatar Aug 26 '20 16:08 PiotrZierhoffer

FYI - There is a manually generated spreadsheet at https://docs.google.com/spreadsheets/d/1KMZ215M_7YQb0l-vCyVdzSsRzaSJN4VVGsTwf4wFK3k/edit#gid=139199577

mithro avatar Oct 22 '20 18:10 mithro

@PiotrZierhoffer - This pull request needs a rebase before I can merge it.

mithro avatar Oct 22 '20 18:10 mithro

The PR has been rebased @mithro PTAL

pkatarzynski avatar Nov 03 '20 11:11 pkatarzynski

At the moment, you don't get any output in the documentation. We need to figure out how to make run on readthedocs (or an alternative way to push things to readthedocs).

mithro avatar Nov 03 '20 16:11 mithro

We would like to add a local sphinx extension to generate the list of cells for each library.

pkatarzynski avatar Nov 04 '20 11:11 pkatarzynski

@pkatarzynski -- If we can figure out the right way to make the ReadTheDocs builder clone the submodules which contain the cells, they should just all start appearing in the documentation? One potential issue is that the builder might run out of disk space...

mithro avatar Nov 04 '20 17:11 mithro

Another option is to run the tool as part of the generation of the library repositories?

mithro avatar Nov 04 '20 19:11 mithro

I have rebased this and resolved conflicts. @proppy can we land this?

kgugala avatar May 20 '22 14:05 kgugala

@mithro were the concern you had back in https://github.com/google/skywater-pdk/pull/143#issuecomment-721942430 addressed by follow up commits: https://github.com/google/skywater-pdk/pull/143/commits/b211454ddb4b0d6f8b145da94924c2dafa30d35e https://github.com/google/skywater-pdk/pull/143/commits/8e9a17a8659352ce5fdee81a89d704a8fc8cb76e ?

proppy avatar May 24 '22 13:05 proppy

@kgugala - Should this index be appearing in the docs @ https://skywater-pdk--143.org.readthedocs.build/en/143/ somewhere?

mithro avatar May 24 '22 15:05 mithro

@kgugala do you know if readthedocs got updated as part of your rebase, or does that needs to be done separatly (if the later, can you share the instructions?)

proppy avatar May 24 '22 15:05 proppy

@proppy the RTD build was/is triggered automatically. See the checks below (click on "Show all checks"). That will take you to the link that @mithro posted. On the bottom right, there is a floating box with a link to https://readthedocs.org/projects/skywater-pdk/builds/.

umarcor avatar May 24 '22 15:05 umarcor

I was able to find the RTD logs https://readthedocs.org/api/v2/build/16966503.txt thanks to the instructions @umarcor provided.

@kgugala @PiotrZierhoffer there doesn't seems to be any call to the cell_list directive that gets added w/ this PR, was there anything else that was done in https://antmicro-skywater-pdk-docs.readthedocs.io/en/test-submodules-in-rtd/contents/libraries/sky130_fd_sc_hd/README.html in order to populate the table?

proppy avatar May 30 '22 04:05 proppy

Looks like some work was done around this in #266 too.

proppy avatar May 30 '22 04:05 proppy

and #256 too.

proppy avatar May 30 '22 05:05 proppy