Repository navigation
docs: 更新 README 安装、围棋训练与压测说明 - #4
Conversation
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
|
|
||
| 一个 Minecraft 模组，为游戏添加一台便携式游戏机，内含 **32 款**经典小游戏。 | ||
|
|
||
| [安装与使用](#-使用方法) · [围棋 AI 与设置](#围棋-ai-与设置) · [构建与测试](#-构建与测试) · [围棋训练](#围棋训练) |
There was a problem hiding this comment.
WARNING: TOC anchors with Chinese characters may not resolve correctly on GitHub
GitHub's anchor generation for non-ASCII headings (e.g., #-使用方法, #围棋-ai-与设置) produces percent-encoded URLs that can be inconsistent across platforms. Consider using ASCII-only anchors or adding explicit HTML anchors like <a id="installation"></a> before each heading.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
| "mctsIterations": 500, | ||
| "komi": 7.5, | ||
| "gpu": true, | ||
| "modelPath": "D:/models/go.nev" |
There was a problem hiding this comment.
WARNING: Windows-specific path example without cross-platform alternative
The example uses D:/models/go.nev which is Windows-only. Linux/macOS users need guidance on path format (e.g., /home/user/models/go.nev). Since the docs later note paths must not contain spaces, add a cross-platform note here.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
| .\gradlew.bat test --no-daemon --tests '*GoGpuStressTest' --tests '*OpenCLLifecycleRegressionTest' | ||
| ``` | ||
|
|
||
| 2026-10-07，在提交 `5c90d39` 对应代码上完成全量构建与 **227 项测试**，0 失败、0 跳过。RTX 2060 上验证了 24 线程、3,072 次并发推理，12 次 GPU/CPU 训练更新，6 轮 OpenCL 上下文创建与释放，以及批次大小 1/64/65/3/129/2 的扩缩容。新增硬件测试在没有可用 GPU 时会跳过，查看报告中的跳过数量可确认本机是否实际执行了设备测试；这些结果不代表其他显卡、驱动或整局棋力的验证。 |
There was a problem hiding this comment.
WARNING: Potentially misleading test result statement
States "227 项测试，0 失败、0 跳过" but then notes hardware tests skip when GPU unavailable. On machines without GPU (e.g., CI), tests will be skipped. Clarify that "0 跳过" applies only to the RTX 2060 validation run, not universally.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
| ```json | ||
| { | ||
| "go": { | ||
| "searchTime": 1000, |
There was a problem hiding this comment.
WARNING: Inconsistent searchTime defaults between JSON config, in-game AI, and training CLI
- JSON example:
searchTime: 1000 - Training CLI table default:
--searchTime 300 - In-game AI (MCTSGoAI.DEFAULT_SEARCH_TIME): 3000ms
These are three different values for conceptually related settings. Document which default applies where, or align the example with the in-game default (3000).
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
| { | ||
| "go": { | ||
| "searchTime": 1000, | ||
| "mctsIterations": 500, |
There was a problem hiding this comment.
WARNING: Naming inconsistency: JSON uses mctsIterations but CLI uses --iterations
The JSON config key is mctsIterations (read by MCTSGoAI line 170) while the training CLI parameter is --iterations (parsed into maxIterations in GoSelfPlayTrainer). They map to the same internal value but the naming difference will confuse users. Document the mapping or use consistent naming.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
|
|
||
| # 产物位于 build/libs/Game Console-1.0.0-NeoForge-1.21.1-beta5.jar | ||
| ```powershell | ||
| .\gradlew.bat test --no-daemon --tests '*GoGpuStressTest' --tests '*OpenCLLifecycleRegressionTest' |
There was a problem hiding this comment.
SUGGESTION: Missing Linux/macOS variant for GPU stress test command
Only PowerShell command shown for GoGpuStressTest and OpenCLLifecycleRegressionTest. Add bash equivalent for parity with the build commands above.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
|
|
||
| ## 围棋训练 | ||
|
|
||
| 训练任务从源码目录运行，无需启动 Minecraft。以下使用 PowerShell；Linux / macOS 将 `.\gradlew.bat` 替换为 `bash ./gradlew`。当前任务按空白拆分 `trainArgs` / `advArgs`，参数中的文件路径请使用不含空格的路径。 |
There was a problem hiding this comment.
WARNING: Path spacing limitation buried in training section
"参数中的文件路径请使用不含空格的路径" is a significant limitation (Gradle splits args on whitespace). This should be highlighted earlier (e.g., in the JSON config section) and mentioned for --weights, --katago, --katagoModel, --katagoConfig paths.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
|
|
||
| `searchTime` 单位为毫秒。导入后重新创建围棋对局以加载模型和搜索设置；修改 GPU 开关后重启客户端。没有有效模型文件时，内置 AI 使用随机初始化的网络权重，训练效果和棋力需要另行评估。 | ||
|
|
||
| GPU 加速需要驱动提供支持双精度运算的 OpenCL 设备。GPU 不可用时自动回退 CPU；初始化失败后会在冷却 30 秒后的下一次尝试中重新探测。模组 JAR 不内嵌 JNA，游戏运行时使用环境提供的 JNA；Gradle 的训练和测试任务会添加所需运行依赖。 |
There was a problem hiding this comment.
WARNING: JNA not bundled in mod JAR - runtime dependency on user environment
"模组 JAR 不内嵌 JNA，游戏运行时使用环境提供的 JNA" means users without JNA in their classpath will silently fall back to CPU. This should be called out more prominently as a known limitation/installation requirement for GPU acceleration.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
| 1. 确保已安装 [NeoForge](https://neoforged.net/) 1.21.1 | ||
| 2. 将 `Game Console-1.0.0-NeoForge-1.21.1-beta5.jar` 放入 `.minecraft/mods` 文件夹 | ||
| 3. 启动游戏，在创造模式物品栏「游戏机」分类中获取游戏机 | ||
| 1. 使用 Java 21，安装 Minecraft 1.21.1 和 [NeoForge](https://neoforged.net/)；当前构建使用 NeoForge 21.1.241。 |
There was a problem hiding this comment.
SUGGESTION: Hardcoded NeoForge version will become outdated
"当前构建使用 NeoForge 21.1.241" duplicates neo_version from gradle.properties. Consider referencing the property or stating "see build.gradle for current version" to avoid drift.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
| 另行准备 [KataGo](https://github.com/lightvector/KataGo) 可执行文件、兼容的模型和 GTP 配置，修改以下示例路径： | ||
|
|
||
| ```powershell | ||
| .\gradlew.bat adversarialTrain --no-daemon "-PadvArgs=--weights data/go.nev --katago D:/katago/katago.exe --katagoModel D:/katago/model.bin.gz --katagoConfig D:/katago/gtp.cfg --games 4 --parallelism 2 --generations 5 --epochs 1 --learningRate 0.001 --gpu true" |
There was a problem hiding this comment.
SUGGESTION: adversarialTrain example uses non-default parameter values
Example shows --games 4 --parallelism 2 --generations 5 but text states "对抗训练默认每代 10 局、并行 5 局". Either update the example to match defaults or explicitly note these are reduced for demo purposes.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
Code Review SummaryStatus: 10 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
SUGGESTION
Files Reviewed (1 file)
Fix these issues in Kilo Cloud Reviewed by free · Input: 580.6K · Output: 10.2K · Cached: 509.8K |
README 的安装示例仍引用 beta5，合成配方多写了两块铁锭，也缺少围棋模型、GPU 和训练任务的使用说明。本文档按当前源码及成品 JAR 修正这些内容，补充可复制的命令、参数默认值和最近的验证结果。
5c90d39的 227 项测试和 RTX 2060 压测结果，说明硬件测试跳过规则与验证范围。验证：已核对源码中的任务名称、示例参数、默认值，解析 JSON 示例，检查本地链接、Markdown 代码块、游戏数量及
git diff --cached --check；合成配方与产物名已对照构建 JAR。此提交仅修改 README，未重复运行代码测试。