📝 Architecture documentation - #795
Merged
Merged
Conversation
MaxieHelenBichmann
marked this pull request as ready for review
August 14, 2026 12:24
Set a global `nb_execution_timeout` of 300s. The MyST-NB default of 30s is too tight for the Monte-Carlo simulations in StatePrep.md, which is what made the Read the Docs build fail intermittently. Drop the now-redundant per-file `execution_timeout` overrides in CatStates.md and Encoders.md; ExactSynthesis.md and cococo.md were at risk on slow builders and are covered now too. Fold state preparation into the Circuit Synthesis category and rename Compilation to Logical Compilation. Give each category landing page a grid of cards instead of a bare toctree, and update the landing page intro to route readers through the new sections. Link the abstractions in the architecture page to their API reference entries, and rename Architecture.md/tools/decoder.md to architecture.md/tools/decoders.md for consistency with the other pages. Fix the dangling `installation.md#optional-features` anchor in StatePrep.md; installation.md is generated from an external template and has no such section, so the qsample extra is now named inline. Assisted-by: Claude Opus 5 via Claude Code
Collaborator
|
I renamed and re-ordered the sections a little: state preparation now lives under Circuit Synthesis rather than being its own category, and Compilation is now Logical Compilation. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes #784
Assisted-by: Codex for build pipeline summarization, first reviewer
Checklist
If PR contains AI-assisted content:
🤖 *AI text below* 🤖(titles are exempt).