Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds a MkDocs-based documentation site (built on Read the Docs) and updates existing user-facing docs to improve clarity around installation, configuration, and pipeline usage.
Changes:
- Introduces MkDocs + Material configuration, RTD build config, and custom docs styling/assets.
- Adds API reference pages for key pipeline components and includes recipe/README content into the docs site.
- Updates CLI/README/recipes to align option help text and usage examples with current behavior.
Reviewed changes
Copilot reviewed 18 out of 24 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/cocoa/cli.py | Updates package version lookup and clarifies CLI help text for tokenizer reuse and verbosity. |
| recipes/date-based-generative-inference.md | Refreshes the recipe instructions and command example for running the pipeline with config overrides. |
| recipes/configure-new-token.md | Clarifies how to apply collation config changes and how to pass the config via CLI. |
| README.md | Improves installation instructions and updates documentation links and option descriptions. |
| pyproject.toml | Adds documentation-related optional dependencies and expands project URLs. |
| mkdocs.yml | Adds MkDocs site configuration, navigation, and plugin setup for API/recipe content. |
| docs/svg/emoji_u2615.svg | Adds a documentation logo/favicon asset for the MkDocs theme. |
| docs/stylesheets/extra.css | Adds custom styling and self-hosted font configuration for the docs theme. |
| docs/recipes/tokenizer-transfer.md | Adds a docs wrapper page that includes the existing recipe content. |
| docs/recipes/index.md | Adds a docs landing page that includes the recipes README content. |
| docs/recipes/date-based-generative-inference.md | Adds a docs wrapper page that includes the existing recipe content. |
| docs/recipes/configure-new-token.md | Adds a docs wrapper page that includes the existing recipe content. |
| docs/index.md | Includes the repository README directly into the docs home page. |
| docs/api/winnower.md | Adds API documentation for the Winnower stage and its outputs/behavior. |
| docs/api/tokenizer.md | Adds API documentation for the Tokenizer stage and tokenizer reuse semantics. |
| docs/api/collator.md | Adds API documentation for the Collator stage and config/security notes. |
| docs/api/cli.md | Adds CLI documentation describing commands and typical usage patterns. |
| .vscode/settings.json | Sets Prettier as the default formatter for CSS files in VSCode. |
| .readthedocs.yaml | Configures Read the Docs to build the MkDocs site using Python 3.12. |
| .gitignore | Updates ignored local/build artifacts (docs build output, dist/build, etc.). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| Collation is driven by a YAML config (the package ships a default; see | ||
| [`./src/cocoa/config/collation.yaml`](./src/cocoa/config/collation.yaml)) that | ||
| specifies: | ||
| [`./src/cocoa/config/collation.yaml`](https://github.com/bbj-lab/cocoa/blob/master/src/cocoa/config/collation.yaml)) |
|
|
||
| Tokenization is driven by its own YAML config (the package ships a default; see | ||
| [`./src/cocoa/config/tokenization.yaml`](./src/cocoa/config/tokenization.yaml)) | ||
| [`./src/cocoa/config/tokenization.yaml`](https://github.com/bbj-lab/cocoa/blob/master/src/cocoa/config/tokenization.yaml)) |
| Winnowing is driven by a YAML config (the package ships a default; see | ||
| [`./src/cocoa/config/winnowing.yaml`](./src/cocoa/config/winnowing.yaml)) that | ||
| specifies: | ||
| [`./src/cocoa/config/winnowing.yaml`](https://github.com/bbj-lab/cocoa/blob/master/src/cocoa/config/winnowing.yaml)) |
Comment on lines
+23
to
+24
| 3. Run the pipeline, passing your configs with `-c`/`--*-config` (any stage | ||
| without an override uses the shipped default): |
| Repository = "https://github.com/bbj-lab/cocoa" | ||
|
|
||
| [project.optional-dependencies] | ||
| all = [ "cocoa-tokenizer[dev,docs]" ] |
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.
This pull request introduces significant improvements to documentation and developer experience, focusing on adding a robust documentation build system, enhancing the
README.mdfor clarity and accuracy, and providing detailed API documentation for key pipeline components. The changes also include custom styling for documentation and improved VSCode settings for CSS formatting.Documentation infrastructure and content:
.readthedocs.yamlconfiguration to enable automated documentation builds with Read the Docs, specifying Python 3.12 and integration withmkdocs.docs/api/cli.md), Collator (docs/api/collator.md), Tokenizer (docs/api/tokenizer.md), and Winnower (docs/api/winnower.md), providing detailed explanations of each pipeline stage and their outputs. [1] [2] [3] [4]README.mdcontent directly in the documentation index for a unified docs experience. [1] [2] [3] [4] [5]README and usage improvements:
README.mdto clarify installation instructions, improve command-line option descriptions, fix and clarify links to configuration files and recipes, and update file exclusion patterns for data syncing. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]Developer experience and styling:
extra.cssstylesheet for documentation, including self-hosted Gotham fonts and a custom color scheme for Material for MkDocs.