mod6 accuracy fixes to agent_safety notebook lecture cells#24
Open
PicoNVIDIA wants to merge 1 commit into
Open
mod6 accuracy fixes to agent_safety notebook lecture cells#24PicoNVIDIA wants to merge 1 commit into
PicoNVIDIA wants to merge 1 commit into
Conversation
Targets the lecture (markdown) cells in code/6-agent-safety/agent_safety.ipynb and its answers twin. Code cells, exercises, and policy files are untouched. Accuracy: - Section 1: M4 framing — Module 4 is Agent Customization; the application-level controls (HITL, allowlists) come from M4's bash agent specifically, not from M4 as a whole. - Section 1: drop "agent drift — static allowlists become stale" triple-em-dash in favor of a flatter sentence. - Section 1 closing print: "OpenShell restricts the PATH, not the command" was inaccurate — Landlock restricts file access, not $PATH. Reworded. - Section 2: name the actual kernel mechanisms (Landlock + seccomp) instead of the vague "enforces security at the kernel level". - Section 3: Privacy Router routes prompts/documents the agent sends to a model, not "every piece of data the agent processes". Also drop the "Local Nemotron (designed to stay within your infrastructure)" framing — the router enforces locality, not the model. Drop "(for best performance)" framing on the public route. - Cell 32: drop the unverifiable internal path `nemoclaw-blueprint/policies/openclaw-sandbox.yaml`. - Cell 34 workshop-arc table: M1-M3 weren't security layers; their column entries are now "—" so the layered story honestly starts at M4. Tone: - Cell 32 lede: "Everything you've built maps directly to NVIDIA's NemoClaw stack" was sales-y; reworded to "Each exercise mirrors a piece of...". - Cell 34 closer: replaced "Each level of capability demands a corresponding level of security. Module 6 closes the loop: your autonomous agent is not just contained — it's evaluated, tested, and continuously verified" with a tighter version. Items still open (not addressed in this commit, need your judgment): 1. OpenShell vs OpenClaw vs NemoClaw — relationship is never explained in the notebook; first-time readers can't tell what's the schema, runtime, or stack name. 2. External links in cell 34 (NVIDIA/NemoClaw, NVIDIA/OpenShell, docs.openclaw.ai) — verify these resolve before shipping. 3. Judge model id "nvidia/nemotron-3-super-120b-a12b" — confirm it resolves on integrate.api.nvidia.com (cells 24-26 silently fail otherwise). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.
for some reason my auth was weird on GH so I had claude fix it, LMK if everything LGTY