Skip to content

docs: tidy Uninstall section - #303

Merged
marianogappa merged 1 commit into
mainfrom
docs/tidy-uninstall
Jul 11, 2026
Merged

docs: tidy Uninstall section#303
marianogappa merged 1 commit into
mainfrom
docs/tidy-uninstall

Conversation

@marianogappa

Copy link
Copy Markdown
Owner

Reorder Uninstall data-folder commands Windows-first in a single code block and trim the section's prose.

@github-actions

Copy link
Copy Markdown
Contributor

🔻 Coverage below the badge

make coverage reports 80.7% (hand-written code; generated code excluded), below the README badge of 81%.

Either add tests, or update the badge: scripts/update-readme-coverage.sh 80.7.

@github-actions

Copy link
Copy Markdown
Contributor

📊 Ingestion benchmark (base → PR)

Corpus: internal/patterns/markers/testdata/replays · 3 samples · GitHub-hosted 2-core runner. Informational only — runner variance is ±10–20%, so a small delta is noise.

goos: linux
goarch: amd64
pkg: github.com/marianogappa/screpdb/internal/storage
cpu: AMD EPYC 7763 64-Core Processor                
                        │ bench-base.txt │       bench-current.txt        │
                        │     sec/op     │   sec/op     vs base           │
SQLiteIngestionCorpus-4      33.83 ± ∞ ¹   34.23 ± ∞ ¹  ~ (p=0.400 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

                        │ bench-base.txt │       bench-current.txt        │
                        │     s/cmds     │   s/cmds     vs base           │
SQLiteIngestionCorpus-4      26.31 ± ∞ ¹   26.65 ± ∞ ¹  ~ (p=0.100 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

                        │ bench-base.txt │        bench-current.txt        │
                        │    s/commit    │   s/commit    vs base           │
SQLiteIngestionCorpus-4     335.1m ± ∞ ¹   346.3m ± ∞ ¹  ~ (p=0.200 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

                        │ bench-base.txt │        bench-current.txt        │
                        │    s/events    │   s/events    vs base           │
SQLiteIngestionCorpus-4     127.6m ± ∞ ¹   125.5m ± ∞ ¹  ~ (p=0.700 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

                        │   bench-base.txt    │           bench-current.txt            │
                        │ s/ingest_166replays │ s/ingest_166replays  vs base           │
SQLiteIngestionCorpus-4           33.80 ± ∞ ¹           34.19 ± ∞ ¹  ~ (p=0.400 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

                        │ bench-base.txt │       bench-current.txt        │
                        │    s/parse     │   s/parse    vs base           │
SQLiteIngestionCorpus-4      5.022 ± ∞ ¹   5.104 ± ∞ ¹  ~ (p=0.400 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

                        │ bench-base.txt │        bench-current.txt        │
                        │   s/patterns   │  s/patterns   vs base           │
SQLiteIngestionCorpus-4     97.25m ± ∞ ¹   93.41m ± ∞ ¹  ~ (p=0.200 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

                        │ bench-base.txt │        bench-current.txt        │
                        │   s/players    │  s/players    vs base           │
SQLiteIngestionCorpus-4     48.89m ± ∞ ¹   48.81m ± ∞ ¹  ~ (p=1.000 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

                        │ bench-base.txt │        bench-current.txt        │
                        │    s/replay    │   s/replay    vs base           │
SQLiteIngestionCorpus-4     203.6m ± ∞ ¹   206.0m ± ∞ ¹  ~ (p=0.400 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

                        │ bench-base.txt │        bench-current.txt        │
                        │  s/replay_ins  │ s/replay_ins  vs base           │
SQLiteIngestionCorpus-4     33.58m ± ∞ ¹   33.64m ± ∞ ¹  ~ (p=0.700 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

                        │ bench-base.txt │        bench-current.txt         │
                        │      B/op      │     B/op       vs base           │
SQLiteIngestionCorpus-4    2.531Gi ± ∞ ¹   2.530Gi ± ∞ ¹  ~ (p=1.000 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

                        │ bench-base.txt │        bench-current.txt        │
                        │   allocs/op    │  allocs/op    vs base           │
SQLiteIngestionCorpus-4     22.34M ± ∞ ¹   22.34M ± ∞ ¹  ~ (p=1.000 n=3) ²
¹ need >= 6 samples for confidence interval at level 0.95
² need >= 4 samples to detect a difference at alpha level 0.05

@marianogappa
marianogappa merged commit e83a995 into main Jul 11, 2026
4 checks passed
@marianogappa
marianogappa deleted the docs/tidy-uninstall branch July 11, 2026 08:18
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.

1 participant