Repository navigation
docs(unit1): move notebook run instructions to Dummy Agent Library section - #731
Open
ammar-iitm wants to merge 1 commit into
Open
ammar-iitm wants to merge 1 commit into
ammar-iitm wants to merge 1 commit into
Conversation
…ction The "What are LLMs?" section ended with instructions to run a notebook (HF token, Llama model access) that isn't introduced until the later "Dummy Agent Library" section, confusing readers before they get there. The Llama access instruction was also stale: the notebook now uses moonshotai/Kimi-K2.5, not a gated Meta Llama model. Moved the still- relevant HF token note to where the notebook is actually linked, and dropped the outdated Llama access line. Fixes huggingface#641 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0128Hr7KB82qLio3BdB3KAyF
zenguos
added a commit
to zenguos/agents-course
that referenced
this pull request
Oct 16, 2026
…ry section Fixes huggingface#641 for the French version (mirrors huggingface#731 for EN). units/fr/unit1/what-are-llms.mdx ended with a misplaced block (HF token + access to the Meta Llama models) although that page has no notebook section.
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
1 participant
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.
Summary
Fixes #641.
The "What are LLMs?" section ended with instructions to run a notebook (get an HF token, request access to a Llama model) that isn't introduced until the much later "Dummy Agent Library" section — confusing for readers since nothing about running a notebook has been mentioned yet at that point.
The Llama access instruction was also stale:
dummy-agent-library.mdxnow usesmoonshotai/Kimi-K2.5via the Serverless API rather than a gatedmeta-llamamodel, so telling every reader they need to request Llama access was simply incorrect.Changes
units/en/unit1/what-are-llms.mdx.units/en/unit1/dummy-agent-library.mdx, right after the notebook is actually linked. Dropped the outdated Llama access line since it no longer applies.Testing
Documentation-only change (English unit1 content). No code/tests affected.
🤖 Generated with Claude Code
https://claude.ai/code/session_0128Hr7KB82qLio3BdB3KAyF