Skip to content

Commit

Permalink
fix the documentation build warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
Krastanov committed Jan 22, 2024
1 parent caf207e commit 52d54a2
Show file tree
Hide file tree
Showing 4 changed files with 17 additions and 4 deletions.
3 changes: 1 addition & 2 deletions docs/make.jl
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,7 @@ clean = true,
sitename = "QuantumClifford.jl",
format = Documenter.HTML(size_threshold_ignore = ["API.md"]),
modules = [QuantumClifford, QuantumClifford.Experimental.NoisyCircuits, QuantumClifford.ECC, QuantumInterface],
warnonly = true, # uncomment if you are having trouble building the docs and are debugging
#warnonly = [:missing_docs],
warnonly = [:missing_docs],
authors = "Stefan Krastanov",
pages = [
"QuantumClifford.jl" => "index.md",
Expand Down
8 changes: 8 additions & 0 deletions docs/src/ECC_evaluating.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,13 @@
# [Evaluating an ECC code and decoders](@id ecc_evaluating)

```@meta
DocTestSetup = quote
using QuantumClifford
using QuantumClifford.ECC
end
CurrentModule = QuantumClifford.ECC
```

!!! warning "The documentation is incomplete"
While waiting for a better documentation than the small example below, consider looking into [`evaluate_decoder`](@ref), [`TableDecoder`](@ref), [`BeliefPropDecoder`](@ref), [`PyBeliefPropDecoder`](@ref), [`PyMatchingDecoder`](@ref), [`CommutationCheckECCSetup`](@ref), [`NaiveSyndromeECCSetup`](@ref), [`ShorSyndromeECCSetup`](@ref)

Expand Down
6 changes: 6 additions & 0 deletions docs/src/references.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,12 @@ These publications describe the uniform sampling of random stabilizer states:
- [berg2020simple](@cite)
- [li2019measurement](@cite)

For circuit construction routines (for stabilizer measurements for a given code):
- [cleve1997efficient](@cite)
- [gottesman1997stabilizer](@cite) (and its erratum)
- [grassl2002algorithmic](@cite)
- [grassl2011variations](@cite)

# References

```@bibliography
Expand Down
4 changes: 2 additions & 2 deletions src/ecc/circuits.jl
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@ The initial physical qubits to be encoded have to be at indices `n-k+1:n`.
Instead, you should measure the stabilizers of the code and the logical observables,
thus projecting into the code space (which can be fault-tolerant).
The canonicalization operation performed on the code may permute the qubits (see [canonicalize_gott!](@ref)).
The canonicalization operation performed on the code may permute the qubits (see [`canonicalize_gott!`](@ref)).
That permutation is corrected for with SWAP gates by default (controlled by the `undoperm` keyword argument).
Based on [gottesman1997stabilizer](@cite) and [cleve1997efficient](@cite),
Based on [cleve1997efficient](@cite) and [gottesman1997stabilizer](@cite),
however it seems the published algorithm has some errors.
Consult the erratum, as well as the more recent [grassl2002algorithmic](@cite) and [grassl2011variations](@cite),
and be aware that this implementation also uses H instead of Z gates.
Expand Down

0 comments on commit 52d54a2

Please sign in to comment.