Repository navigation
docs: overhaul documentation for external readers - #205
Merged
Merged
Conversation
Mark the agent, optimize, and distill extras as experimental on every surface a reader meets them on: README feature list and extras table, docs/index.md, docs/installation.md extras table, docs/cli.md command index and per-command admonitions, docs/agent.md top matter, examples/agent/README.md, and llms.txt. Each marking states the API may change without notice or a deprecation cycle, and names the core pipeline as the stable surface. Co-authored-by: Gwenlyn Glusman <3977332+gglusman@users.noreply.github.com>
README gains a short Statement of need (the problem, the audience, and the relation to per-source Python ingests and hosted name resolution) and a Getting help section pointing at the issue tracker and CONTRIBUTING.md. docs/index.md carries the same need and audience in its own words plus the support link. CONTRIBUTING.md folds Reporting issues into a Getting help section that covers usage questions, bugs and feature requests, documentation drift, and private security reports. Co-authored-by: Gwenlyn Glusman <3977332+gglusman@users.noreply.github.com>
README stops repeating the per-extra install matrix in prose and compresses the missing-extra paragraph into a pointer at the installation guide, which owns the full matrix and per-command preflight behavior. installation.md drops 57 lines of duplication: the repeated per-extra install command block becomes one generic pattern, the four install methods become three (PyPI, GitHub main, development from source, in that order so users hit PyPI first), and the Development Setup / Upgrading sections collapse into pointers at development.md and CONTRIBUTING.md. Wheels-vs-sdist platform coverage and the Rust toolchain requirement are now stated where the install methods are. The experimental agent/optimize/distill surfaces are named in the opening gloss. Co-authored-by: Gwenlyn Glusman <3977332+gglusman@users.noreply.github.com>
The tutorial named fullmap as a prerequisite without saying how to get it, so a first run failed at Step 4 with no path forward. Prerequisites now carry the build-fullmap command, the --output path the graph config expects, a link to the fullmap guide, and an honest time note that obtaining the database is a separate multi-GB step. Step 3's replacement note names the same path, and the README quick start now points at ./fullmap with the build step called out instead of an unexplained /path/to/fullmap. llms.txt's Installation Guide entry drops the per-extra gloss the installation guide already owns and names the experimental extras. Co-authored-by: Gwenlyn Glusman <3977332+gglusman@users.noreply.github.com>
Replace every em dash, en dash, ellipsis character, arrow glyph, smart quote, and math glyph in the live documentation prose with plain ASCII (->, =>, <->, ..., -, <=, >=, x). Appositive em-dash pairs become parentheses, semicolons, or colons as the sentence requires; comma splices introduced by the mechanical pass are repaired by hand. Code blocks keep their alignment; only the glyphs inside them change. The two remaining non-ASCII glyphs are the verbatim program output quoted in docs/cli.md (the progress stage line prints them) and author-name accents in the README. CONTRIBUTING.md also folds Reporting issues into a Getting help section that covers usage questions, bug and feature reports, documentation drift, and private security reports. Co-authored-by: Gwenlyn Glusman <3977332+gglusman@users.noreply.github.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
SkyeAv
added a commit
that referenced
this pull request
Oct 1, 2026
Cut 19.5.1 and bump the package version in pyproject.toml, CITATION.cff, and uv.lock. Patch: docs-only surface refresh plus a behavior-preserving dependency bump. biolink-model 4.4.5 absorbs the dropped CLASS_FIELD_OVERRIDES grants upstream (the effective field set on the pinned EntityToDiseaseAssociation / EntityToPhenotypicFeatureAssociation classes is unchanged) and the Rust layer passes validation_ctx by value to satisfy Rust 1.99's needless_borrows_for_generic_args lint (#209). Documentation gains a statement of need and audience, a Getting help pathway, experimental labels on the agent/optimize/distill surfaces, runnable fullmap prerequisites in the tutorial, a three-method installation guide, and plain-ASCII prose (#205); CODE_OF_CONDUCT.md ships Contributor Covenant 2.1 with CITATION.cff maintainer emails as enforcement contacts (#206); CITATION.cff gains verified ORCIDs for all three authors and a factual abstract (#207). Changelog: - Versioned the release as 19.5.1 dated 2026-10-01; added Changed entries for #205, #206, #207, and #209 with PR links. Docs: none needed here; the release changelog section is the docs update. Testing: - make check -> exit 0: ruff check/format clean, pyright clean, pytest 1674 passed / 52 skipped in 29.33s, cargo test 156 passed / 0 failed (127 + 11 + 17 + 1 across 5 suites), cargo clippy -D warnings clean
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.
Documentation-only overhaul aimed at external readers evaluating the project cold: an explicit statement of need, a support pathway, experimental labels on the agent/distill surfaces, runnable tutorial prerequisites, less duplication, and plain-ASCII prose. No code, tests, or config behavior changes; the diff touches only
.mdfiles andllms.txt(18 files, +343/-308).Statement of need and audience
## Statement of need: the problem (per-source Python ingests plus hosted name resolution for tabular biomedical sources), the audience (Translator ingest authors, KG data engineers, bioinformatics groups), and the relation toNCATSTranslator/translator-ingestspractice including RIG emission.docs/index.md## Who this is for: the same framing in its own words (the nav-parity guard requires the page lists stay intact, so this is additive prose only).Support pathway
## Getting help: issues for questions (with thequestionlabel) and bugs,CONTRIBUTING.mdfor development, CLI/configuration references for flag lookup.CONTRIBUTING.md## Getting help: replaces the thinner "Reporting issues" section; covers usage questions, bug/feature templates with the details to include, documentation drift (source files named as the authority), and private security reports via the maintainer emails inCITATION.cff. The repo has GitHub private vulnerability reporting disabled, so the email path is the only private channel today.Experimental labeling
docs/index.md,docs/installation.mdintro + extras table,docs/cli.mdcommand index + per-command admonitions (agent,distill-export,distill-weigh),docs/agent.mdtitle + top warning,examples/agent/README.md,llms.txt.build-kg,validate,validate-kgx,build-fullmap,quick-map) is unaffected.Runnable prerequisites
tablassert build-fullmap --output ./fullmap/data/fullmap.redbcommand, thefullmap:value the graph config expects, a link to the fullmap guide, and an honest time note that obtaining the database is a separate multi-GB step (a reader following the tutorial literally previously failed at Step 4 with no path forward). Step 3's replacement note names the same path.fullmap: /path/to/fullmapbecomesfullmap: ./fullmapwith the build step called out and linked.Concision
docs/installation.mdowns; the missing-extra paragraph compresses to the failure contract plus a pointer.docs/installation.md(-57 net lines): four install methods become three in user-first order (PyPI, GitHub main, development from source); the per-extra command block becomes one generic pattern plus a combine example; Development Setup/Upgrading collapse into pointers atdocs/development.mdandCONTRIBUTING.md; wheels (Linux/macOS) vs sdist (Rust toolchain) coverage is stated where the methods are.llms.txt: the Installation Guide entry drops per-extra glosses the installation guide already owns and names the experimental extras.Plain ASCII prose
->,=>,<->,...,-,<=,>=,x). Appositive em-dash pairs become parentheses, colons, or semicolons as the sentence requires; no comma splices left behind. Code blocks keep their alignment (only glyphs inside them change).✓ Stage N · NAME · elapsedprogress line indocs/cli.md(verbatim program output fromsrc/tablassert/progress.py) and author-name accents in the README.Deferred
CITATION.cff: not added; the identifiers are the authors' to supply.pyproject.tomlsummary/description wording: untouched to keep the diff docs-only.Testing
make check-> exit 0:ruff checkall passed,ruff format --checkclean,pyright0 errors, 0 warnings, 0 informations,uv run pytest -n auto->1674 passed, 52 skipped,cargo fmt --checkclean,cargo test->127 passed+11 passed, 1 ignored+17 passed+1 passed,cargo clippy --all-targets -- -D warningscleanuv run pytest -q --no-cov tests/test_docs_source_of_truth.py tests/test_docs_examples.py tests/test_docs_cli_coverage.py tests/test_agent_docs.py->233 passed(run at every commit)uv run mkdocs build --strict -q-> exit 0 (run at every commit)git diff --name-only f5b717f..HEAD | rg -v '\.md$|^llms\.txt$'-> empty (docs-only scope confirmed)Co-authored-by: Gwenlyn Glusman 3977332+gglusman@users.noreply.github.com