README: lead with the operation counts, and say what is missing rather than what passes - #34
Merged
Merged
Conversation
…r than what passes 529 lines to 491, and the reordering matters more than the length. THE COUNTS LEAD. They are the result of this project and they sat at line 70, below a gate scoreboard, a per-family coverage table and a four-branch exemption note. They are now the first section, with the genus-3 ramified rows -- the new work -- grouped beside their genus-2 counterparts instead of split across the table by model. The `source` column is given the emphasis it needs. *Measured* means verification/opcount.py executed the formulas and identified the frequent case by observing which branch was taken; *published* means read from the thesis, because opcount.py cannot measure split families and refuses rather than guessing. Half the table is therefore not independently checked by this repository, and a reader skimming for numbers has to be told that in the same breath as the numbers. THE SIX-CELL MATRIX IS GONE. It was a progress tracker: its content was the word "missing" in three cells. Every cell ships, so it now says only that a thing is finished, which the counts imply. One sentence replaces the grid. STATUS BECOMES "CURRENT AND PLANNED WORK". The old section was ~93 lines of evidence -- five gate figures, coverage by family, the measured-versus-published note, an operation-count table, a not-runnable line. Evidence belongs beside the commands that reproduce it, so the gate figures moved into Testing with the command for each in its own column. What replaces it is what is missing: the Birkner/GKP comparison the char-2 formulas owe, the split-family blindness in opcount.py and the two proved savings waiting on it, E20's corpus-adequacy gap, the absent positive-reduced basis at genus 3, and the dead converter. That section absorbed "Known gaps and roadmap", which was saying the same thing in a different place -- and WHICH WAS WRONG. Its first bullet read "Genus 3 ramified is arb and nch2 only. The ch2 specialisation is still to be derived, so five of the six cells exist." PR7+8 closed that and the bullet was missed. A README that states a shipped cell is missing is worse than a long one. Every figure re-measured, not copied, because this file has carried stale numbers three times in this series: whitebox 1,886/1,886 and 1,925/1,929, driver 13,746, selftest 17, the suite's 30 testers counted from its own per-tester logs, and all twelve op-count rows re-run through opcount.py -- including the six genus-2 rows that were not being changed, since the point of the gate is that a moved figure is a re-derived one. Two links to the deleted #status anchor were repointed at Testing rather than left dangling, and one bare `opcount.py` became the real path so readme-paths resolves it.
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.
529 lines to 491, and the reordering matters more than the length.
The counts lead
They are the result of this project, and they sat at line 70 — below a gate scoreboard, a per-family
coverage table and a four-branch exemption note. They are now the first section, with the genus-3
ramified rows grouped beside their genus-2 counterparts rather than split across the table by model.
The
sourcecolumn is given the emphasis it needs. Measured meansverification/opcount.pyexecuted the formulas and identified the frequent case by observing which branch was taken.
Published means read from the thesis, because
opcount.pycannot measure split families andrefuses rather than guessing. Half the table is therefore not independently checked by this
repository, and a reader skimming for numbers has to be told that in the same breath as the numbers.
The six-cell matrix is gone
It was a progress tracker: its content was the word "missing" in three cells. Every cell ships now, so
it says only that a thing is finished — which the counts already imply. One sentence replaces the grid.
Status becomes "Current and planned work"
The old section was ~93 of the file's 529 lines and was mostly evidence: five gate figures, coverage
by family, the measured-versus-published note, an operation-count table, a not-runnable line. Evidence
belongs beside the commands that reproduce it, so the gate figures moved into Testing, each with
its command in its own column.
What replaces it is what is missing: the Birkner/GKP comparison the char-2 formulas owe, the
split-family blindness in
opcount.pyand the two proved savings waiting on it, E20'scorpus-adequacy gap, the absent positive-reduced basis at genus 3, and the dead converter.
That section absorbed Known gaps and roadmap, which was saying the same thing in another place —
and which was wrong. Its first bullet read "Genus 3 ramified is
arbandnch2only. Thech2specialisation is still to be derived, so five of the six cells exist." PR7+8 (#32) closed that and
the bullet was missed. A README that states a shipped cell is missing is worse than a long one.
Every figure re-measured, not copied
This file has carried stale numbers three times in this series — tester counts, corpus size, the
matrix row — each time because a figure was moved rather than re-derived. So all of them were re-run:
--strict13,746 compared, 0 wrongrun_familycalls(17 calls is not 34 testers — three are
skip_testpaths)opcount.py, including the six genus-2 rows that werenot being changed, since the point is that a moved figure is a re-derived one
Also
Two links to the deleted
#statusanchor were repointed at Testing rather than left dangling, and onebare
opcount.pybecame the real path soreadme-pathsresolves it.Gates: Magma suite 30 passed, 0 failed, 0 skipped; whitebox 1,886/1,886; selftest 17/0;
readme-pathsandcheck_paths --strictboth clean.