Skip to content
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

CI for building doxygen document and deploy to GitHub pages #5

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

WeileiZeng
Copy link
Member

@WeileiZeng WeileiZeng commented Aug 7, 2024

The page is up on https://qec-pages.github.io/dist-m4ri/index.html

The workflow will run upon push at branches main or doxy. README.md is configured to be the main page.

To develop doc locally, run sudo apt-get install doxygen graphviz -y to install doxygen. Then run doxygen and opendocs/html/index.html in the browser.

Currently, all codes are documented. There are options to hide codes or source files as needed, then only classes will be documented.

@LeonidPryadko
Copy link
Member

Thanks! You can add me as a reviewer (near top-right corner). This way I get a message that I need to look at the code. Of course, I can just do it...

Copy link
Member

@LeonidPryadko LeonidPryadko Aug 7, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Can you please enable mathjax ? There are some equations, and also some code snippets which are supposed to be typed in \tt font but what comes out on my machine is, e.g., "(method=1)" (in a section title)
  • Do you think it would be nice to have pdf output automatically generated?
  • Let us have letter paper, not a4 -- letter can be printed on a4 paper w/o much trouble but not the other way.
  • Somewhere early on, we need to put a reference to QDistRnd GAP package. Saying that it works with more general codes (CSS and F-linear stabilizer codes over an arbitrary field F), as opposed to binary CSS here, but only with an equivalent of method=2.
  • Also, see additional items in documentation #3 : add reference to the JOSS paper on QDistRnd (I can update the text after the first bib file is created)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mathjax has been enabled. However, it doesn't support $single dollar sign$ but \f$ formula \f$. I didn't find a way to incorporate markdown and doxygen. So I made a copy introduction.md as the main page for the doxygen document. I added syntax example in the end of introduction.md

Copy link
Member Author

@WeileiZeng WeileiZeng Aug 8, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the pdf output, I don't think it is good to update pdf frequently in a git repo. Meanwhile, more CI workflow will easily get broken in the long run and become hard to maintain. I would suggest build pdf locally and upload to the release page only when necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants