Skip to content

feat(nemo-agents): add Fabric MCP calculator example#889

Draft
AnuradhaKaruppiah wants to merge 11 commits into
NVIDIA-NeMo:mainfrom
AnuradhaKaruppiah:ak-toy-exammple
Draft

feat(nemo-agents): add Fabric MCP calculator example#889
AnuradhaKaruppiah wants to merge 11 commits into
NVIDIA-NeMo:mainfrom
AnuradhaKaruppiah:ak-toy-exammple

Conversation

@AnuradhaKaruppiah

@AnuradhaKaruppiah AnuradhaKaruppiah commented Jul 24, 2026

Copy link
Copy Markdown

Summary

  • add a standalone fabric-calculator-agent example whose only runtime code is a toy stdio MCP server under mcps/
  • provide two complete Platform-owned configs under configs/: agent.yaml without MCP and agent_with_mcp.yaml with the calculator MCP server
  • enable Relay ATIF and ATOF telemetry and document how to compare the runs and verify calculator tool calls
  • store generated runtime state and telemetry under the example's ignored .tmp/ directory
  • rely on adapter descriptors from the 0.1.0a20260724 Fabric release rather than bundling descriptors

Dependency

This branch includes and depends on #890 for the shared skills, mcp, and tools AgentConfig-to-FabricConfig translation. The example reuses #890's MCP contract (transport plus url) instead of adding a parallel implementation. Once #890 merges, this branch can be rebased so this PR contains only the example diff.

Why

The Fabric invocation path added in #865 needs a small end-to-end example that demonstrates optional MCP configuration without mixing Fabric installation and runtime instructions into the existing NVIDIA NeMo Agent Toolkit calculator example.

Separate full configs keep the comparison explicit without introducing a profile concept: users run one config without tools, run the second with the calculator MCP server, and inspect telemetry to confirm whether the tool was called.

Validation

  • 46 passed across the focused agent config, CLI, Fabric translation, invocation, and calculator example tests
  • verified a real MCP stdio initialize/list-tools/call-tool exchange; multiply([12, 8]) returns 96.0
  • Ruff lint and formatting checks passed for all changed Python files
  • uv lock --check passed
  • verified both live configs start the released Codex adapter and NeMo Relay 0.6.0 with the expected adapter plan; the authenticated Codex account returned Quota exceeded before the model turn, so the model-driven tool call could not complete in this environment

The README includes the commands to rerun both variants and inspect steps[].tool_calls[] in the generated ATIF trajectory.

AjayThorve and others added 8 commits July 22, 2026 23:48
Signed-off-by: Manjesh Mogallapalli <[email protected]>
Signed-off-by: Manjesh Mogallapalli <[email protected]>
Signed-off-by: Manjesh Mogallapalli <[email protected]>
Signed-off-by: Anuradha Karuppiah <[email protected]>

# Conflicts:
#	plugins/nemo-agents/examples/nemo-agent-config/README.md
#	plugins/nemo-agents/tests/unit/test_fabric_translator.py
@github-actions github-actions Bot added the feat label Jul 24, 2026
mmogallapalli and others added 3 commits July 24, 2026 12:06
Signed-off-by: Manjesh Mogallapalli <[email protected]>
Signed-off-by: Anuradha Karuppiah <[email protected]>
@AnuradhaKaruppiah AnuradhaKaruppiah changed the title feat(nemo-agents): add profiled Fabric MCP calculator example feat(nemo-agents): add Fabric MCP calculator example Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants