Skip to content

Reorganize day2 modules: renumber and remove 05-coding-agent - #38

Merged
ShuN6211 merged 2 commits into
mainfrom
feat/day2-renumber-modules
Apr 27, 2026
Merged

Reorganize day2 modules: renumber and remove 05-coding-agent#38
ShuN6211 merged 2 commits into
mainfrom
feat/day2-renumber-modules

Conversation

@ShuN6211

Copy link
Copy Markdown
Collaborator

変更内容

モジュールの削除

  • 05-coding-agent を削除(未使用モジュール)

モジュールの番号振り直し

変更前 変更後
06-attention 01-attention (新しい最初のモジュール)
01-multi-modal 02-multi-modal
02-llm-api 03-llm-api
03-llm-tool 04-llm-tool
04-react-agent 05-react-agent
07-own_agent 06-own_agent (最終モジュール)

技術的修正

  • ruff E402 警告を修正(ファイルレベルの noqa コメントを追加)
    • 05-react-agent/notebook_script.py
    • 06-own_agent/main.py

最終的なモジュール順序

  1. 01-attention: Attention メカニズムの可視化
  2. 02-multi-modal: SigLIP マルチモーダル理解
  3. 03-llm-api: 基本的な LLM API 使用法
  4. 04-llm-tool: Tool calling パターン
  5. 05-react-agent: llama-index を使った ReAct エージェント
  6. 06-own_agent: カスタマイズ可能なベースラインエージェント

動作確認

05-react-agent

cd day2/05-react-agent
uv sync
uv run test_quick.py
# Query: What is 20+(2*4)?
# Answer: 28

06-own_agent

cd day2/06-own_agent
uv sync
uv run main.py
# Query: What is 20 + (2 * 4)? Calculate step by step.
# Answer: 28

レビューポイント

  • モジュール番号の変更が適切
  • 05-coding-agent の削除が問題ない
  • ruff lint が通る
  • 動作確認済み

🤖 Generated with Claude Code

ShuN6211 and others added 2 commits April 27, 2026 22:52
Changes:
- Delete 05-coding-agent (unused module)
- Renumber modules:
  - 06-attention → 01-attention (new first module)
  - 01-multi-modal → 02-multi-modal
  - 02-llm-api → 03-llm-api
  - 03-llm-tool → 04-llm-tool
  - 04-react-agent → 05-react-agent
  - 07-own_agent → 06-own_agent (final module)
- Fix ruff E402 warnings with file-level noqa comments

Final module order:
1. 01-attention: Attention mechanism visualization
2. 02-multi-modal: SigLIP multimodal understanding
3. 03-llm-api: Basic LLM API usage
4. 04-llm-tool: Tool calling patterns
5. 05-react-agent: ReAct agent with llama-index
6. 06-own_agent: Customizable baseline agent

Verified:
- 05-react-agent: ReActAgent works correctly
- 06-own_agent: main.py executes successfully

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@ShuN6211
ShuN6211 merged commit 8518743 into main Apr 27, 2026
7 checks passed
@ShuN6211
ShuN6211 deleted the feat/day2-renumber-modules branch April 27, 2026 13:56
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