Skip to content

Add proofreading edit history vignette; fix edit cache bugs - #41

Open
jefferis wants to merge 4 commits into
mainfrom
edit-history-vignette
Open

jefferis wants to merge 4 commits into
mainfrom
edit-history-vignette

Conversation

@jefferis

Copy link
Copy Markdown
Member

Adds a vignette plotting the history of human proofreading edits (cumulative and per week, merges vs splits), and fixes two bugs found while building it.

Vignette

  • vignettes/edit-history.Rmd uses aedes:::aedes_all_operations(). Figures are regenerated locally with AEDES_LIVE_VIGNETTES=true (needs CAVE access and a populated edit cache); otherwise the saved PNGs in vignettes/figures/ are shown, including on pkgdown CI.
  • Excludes the automated BANCautomerge account (8041, about 75k nucleus merges in Sep 2025) and failed operations (operation_status != 0).
  • Currently covers 219,858 human edits by 39 users, 2025-02-15 to 2026-10-15.
  • ggplot2 is added to Suggests.

Fixes

  • Multi-chunk fetches stored wrong operation ids. unlist() prefixed ids with the chunk index, so ops fetched in more than one chunk were cached under the wrong id and also recorded as missing. Existing edit caches built this way should be deleted (rappdirs::user_cache_dir("aedes"), *_operations.arrow and *_missing_ops.rds).
  • The last-edit search stopped at gaps in operation ids. Op ids are not contiguous (gaps of up to 19 ids), so the binary search could stop early (322241 rather than 418443). It now checks a window of 100 ids at each step.

🤖 Generated with Claude Code

jefferis and others added 4 commits October 15, 2026 11:50
unlist() prefixed operation ids with the chunk index, so ops fetched in
more than one chunk were stored under the wrong id and recorded as missing
in the edit cache. Existing caches built this way should be deleted.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Plots cumulative and weekly human merges/splits from the local edit cache.
Figures are regenerated locally with AEDES_LIVE_VIGNETTES=true; elsewhere
the saved PNGs are shown.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Operation ids are not contiguous, so probing single ids could stop the
binary search at a gap (e.g. 322241 instead of 418443). Probe a window
of ids instead and return the largest existing id.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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