-
Notifications
You must be signed in to change notification settings - Fork 396
(WIP) Developer documenation of Lc0 #144
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
🚀 Draft Preview: https://pr144-draft.lc0.org/ |
🚀 Draft Preview: https://pr144-draft.lc0.org/ |
🚀 Draft Preview: https://pr144-draft.lc0.org/ |
🚀 Draft Preview: https://pr144-draft.lc0.org/ |
🚀 Draft Preview: https://pr144-draft.lc0.org/ |
🚀 Draft Preview: https://pr144-draft.lc0.org/ |
🚀 Draft Preview: https://pr144-draft.lc0.org/ |
Most sections are still empty (and draft, so don't render), but what's there, I believe is fine to merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces comprehensive developer documentation for the Leela Chess Zero (Lc0) ecosystem, covering the engine architecture, search algorithms, and supporting infrastructure. The documentation provides both high-level overviews and detailed technical specifications for developers working with the codebase.
- Creates new documentation structure covering the entire Lc0 ecosystem including the engine, training pipeline, and supporting services
- Adds detailed technical documentation for the lc3 search algorithm, including worker architecture and data flow
- Provides architectural overview and AlphaZero primer for developers
Reviewed Changes
Copilot reviewed 20 out of 20 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
content/dev/overview.md | Main ecosystem overview documenting all Lc0 components and their relationships |
content/dev/_index.md | Developer documentation landing page with updated structure |
content/dev/lc0/_index.md | Engine-specific documentation introduction |
content/dev/lc0/architecture.md | High-level architecture overview of the engine codebase |
content/dev/lc0/search/_index.md | Search algorithms section introduction |
content/dev/lc0/search/alphazero.md | AlphaZero algorithm primer with mathematical notation |
content/dev/lc0/search/new.md | Guide for implementing new search algorithms |
content/dev/lc0/search/lc3/_index.md | Introduction to the experimental lc3 search algorithm |
content/dev/lc0/search/lc3/overview.md | Detailed lc3 architecture with worker flow diagrams |
content/dev/lc0/search/lc3/policy.md | Search policy framework documentation |
content/dev/lc0/search/lc3/glossary.md | Terminology and data types reference |
Multiple placeholder files | Draft pages for future content sections |
content/dev/old/_index.md | Outdated documents section metadata update |
archetypes/default.md | Template update for new documentation pages |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
||
### Developer Website | ||
|
||
(Python/Django, <https://dev.lczero.org>) A new website in development that is intended to replace the current <https://training.lczero.org> interface, the <https://lc0.org> URL shoretener, and additional services: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a typo in 'shoretener' - it should be 'shortener'.
(Python/Django, <https://dev.lczero.org>) A new website in development that is intended to replace the current <https://training.lczero.org> interface, the <https://lc0.org> URL shoretener, and additional services: | |
(Python/Django, <https://dev.lczero.org>) A new website in development that is intended to replace the current <https://training.lczero.org> interface, the <https://lc0.org> URL shortener, and additional services: |
Copilot uses AI. Check for mistakes.
|
||
It's described in the detail in the papers | ||
|
||
* "[Mastering the game of Go without human knowledge](hhttps://www.nature.com/articles/nature24270.epdf?author_access_token=VJXbVjaSHxFoctQQ4p2k4tRgN0jAjWel9jnR3ZoTv0PVW4gB86EEpGqTRDtpIz-2rmo8-KG06gqVobU5NSCFeHILHcVFUeMsbvwS-lxjqQGg98faovwjxeTUgZAUMnRQ)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The URL has a double 'h' at the beginning - 'hhttps' should be 'https'.
* "[Mastering the game of Go without human knowledge](hhttps://www.nature.com/articles/nature24270.epdf?author_access_token=VJXbVjaSHxFoctQQ4p2k4tRgN0jAjWel9jnR3ZoTv0PVW4gB86EEpGqTRDtpIz-2rmo8-KG06gqVobU5NSCFeHILHcVFUeMsbvwS-lxjqQGg98faovwjxeTUgZAUMnRQ)" | |
* "[Mastering the game of Go without human knowledge](https://www.nature.com/articles/nature24270.epdf?author_access_token=VJXbVjaSHxFoctQQ4p2k4tRgN0jAjWel9jnR3ZoTv0PVW4gB86EEpGqTRDtpIz-2rmo8-KG06gqVobU5NSCFeHILHcVFUeMsbvwS-lxjqQGg98faovwjxeTUgZAUMnRQ)" |
Copilot uses AI. Check for mistakes.
summary: "A description of the lc0 backends" | ||
--- | ||
|
||
Starting with version v0.32, `lc0` defin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sentence is incomplete and cuts off mid-word. This appears to be an unfinished draft that should either be completed or removed.
Starting with version v0.32, `lc0` defin |
Copilot uses AI. Check for mistakes.
No description provided.