Docs: Python workbench coverage + screenshots, em-dash purge, clinical-stack cross-links#81
Merged
Merged
Conversation
…stack - index.qmd's analytics bullet said SQL and R only; the tour never mentioned the Python tab. Both now match the three-tab workbench that shipped in #71. - analytics.qmd gains the Python workbench screenshot slot (image lands with the screenshot refresh). - installation.qmd and README point co-installs (edc-core + ctms-core, shared Keycloak/Caddy/Postgres) at the clinical-stack repo, which already links back here for single-app installs.
Applied the avoid-ai-writing pass: em dashes (previously 8-24 per page) replaced with colons, semicolons, parentheses, or sentence breaks; the one negation-pivot cliche in review.qmd rewritten. Regulatory citations untouched except punctuation; the one rephrased E6(R3) claim (audit-trail review) re-grounded against the source text (§4.2.3).
…-python workbench-sql/r predated the third tab (#71): both showed a two-tab bar and the old 'SQL and R' page copy. Re-shot all three against a fresh v0.4.0-image demo seed, plus a new workbench-python.png (pandas describe + result grid, language badges in the execution history). subject-matrix re-shot from the same stack; MatrixPage changed since the last refresh (#74) but renders identically for an unblinded study. The other 13 screenshots depict pages untouched since the v0.3.0 refresh and are kept.
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.
Stacked on #79 (targets
infra/single-vm-deployment; retarget tomainafter #79 merges — the diff should then show only the three docs commits).What
Three passes over the user-facing docs (site/*.qmd, guide pages, README):
index.qmd's analytics bullet said "real SQL and real R" and the tour only mentioned the R tab. Fixed both, andanalytics.qmdnow shows the Python tab.workbench-sql.pngandworkbench-r.pngpredated the Python tab (two-tab bar, old page copy); both re-shot, plus a newworkbench-python.png(pandasdescribe()output, result grid, language badges in the execution history). Captured from a fresh demo seed on the current images at the same 1440pt@2x full-page framing as the v0.3.0 refresh. The other screenshots depict pages unchanged since that refresh (verified by diffingapps/web/src/pagesagainst the refresh commit; the one exception,MatrixPage, renders identically for an unblinded study — a re-shot came out byte-identical).compliance.qmd); now at most one per page, replaced with colons, semicolons, parentheses, or sentence breaks. One negation-pivot cliché inreview.qmdrewritten. Regulatory citations were restructured only around punctuation; the single rephrased claim (E6(R3) audit-trail review,review.qmd) was re-grounded against §4.2.3 of the guideline text.Also adds the cross-link the repo split implied but never made:
installation.qmdand the README now point co-installs (edc-core + ctms-core, shared Keycloak/Caddy/Postgres) at clinical-stack, which already links back here for single-app installs.Verification
quarto render site/clean (14 pages).