Skip to content

update bittensor.md#4

Open
ping-ke wants to merge 7 commits intomainfrom
bittensor
Open

update bittensor.md#4
ping-ke wants to merge 7 commits intomainfrom
bittensor

Conversation

@ping-ke
Copy link
Copy Markdown
Owner

@ping-ke ping-ke commented Apr 6, 2026

test claude-review

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

This diff is a documentation update to a Bittensor technical presentation file (Chinese). There is no executable code being changed — only markdown content, diagrams, and explanatory text. I'll review it for factual/logical accuracy issues that could mislead readers.


[WARNING] Anchor link mismatch in table of contents

Line -16. [Demo 设计方案](#17-demo-设计方案) was changed to +16. [Demo 设计方案](#17-demo-设计方案). The section was renumbered to 16, but the anchor still points to #17-demo-设计方案. The actual heading is now ## 16. Demo 设计方案, which generates anchor #16-demo-设计方案. This TOC link will be broken.


[WARNING] Trailing whitespace in docker command

-docker run --rm -p 9944:9944 opentensor/subtensor:latest --dev
+docker run --rm -p 9944:9944 opentensor/subtensor:latest --dev 

A trailing space was added. Harmless in most shells, but if this is copy-pasted from a rendered page it may cause subtle issues in some terminal emulators or scripts. Minor, but a regression with no benefit.


[WARNING] Alpha Outstanding upper-bound formula is ambiguous/potentially incorrect

The document states:

③ Alpha Outstanding += min(Δτ̄/Σp_j, 1)  (待分配的 Alpha,上限 1/block)

The formula Δτ̄/Σp_j is unexplained — Δτ̄ (mean/average emission?) and Σp_j (sum of all subnet prices?) are not defined anywhere in the diff. This differs from the narrative description which says the cap is simply 1 alpha/block. If this formula is wrong, readers building on this documentation will implement the emission model incorrectly.


No critical security, concurrency, or logic bugs — this is documentation only.

CRITICAL_COUNT: 0
WARNING_COUNT: 3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant