Skip to content

docs: add optimizer docs hidden from docs build - #1014

Open
jeremysilva1098 wants to merge 2 commits into
mainfrom
optimizer-docs/jesilva
Open

docs: add optimizer docs hidden from docs build#1014
jeremysilva1098 wants to merge 2 commits into
mainfrom
optimizer-docs/jesilva

Conversation

@jeremysilva1098

@jeremysilva1098 jeremysilva1098 commented Jul 31, 2026

Copy link
Copy Markdown

Summary by CodeRabbit

  • Documentation
    • Added comprehensive guidance for Insight-Driven Optimization, including agent roles, prerequisites, setup, workflows, CLI usage, scheduling, configuration, artifacts, and troubleshooting.
    • Documented telemetry-driven analysis, experimentation, validation, evaluation, persistence options, and draft pull request workflows.
    • Added gated navigation entries and publication guidance for the new Agents documentation, making the content easier to discover when released.

Signed-off-by: Jeremy Silva <jesilva@nvidia.com>
@jeremysilva1098
jeremysilva1098 requested review from a team as code owners July 31, 2026 17:24
@github-actions

Copy link
Copy Markdown
Contributor

@coderabbitai

coderabbitai Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Adds a comprehensive Insight-Driven Optimization guide and navigation entries for gated and latest documentation versions.

Changes

Insight-Driven Optimization documentation

Layer / File(s) Summary
Insight-Driven Optimization guide
docs/agents/insight-driven-optimization.mdx
Documents the Analyst, Experimenter, and Eval Author agents, setup, workflows, data persistence, CLI commands, scheduling, evaluation, outputs, and troubleshooting.
Documentation navigation wiring
docs/fern/gated-nav.yml, docs/fern/versions/latest.yml
Adds gated navigation placement and commented guidance for publishing the Agents page in the latest documentation version.

Possibly related PRs

Suggested labels: docs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly describes the addition of optimizer documentation and its exclusion from the docs build.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch optimizer-docs/jesilva

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 3

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/agents/insight-driven-optimization.mdx`:
- Around line 356-357: Update both Studio links in the optimization setup
documentation to construct their URLs from the exported NMP_BASE_URL and
WORKSPACE values instead of hard-coded localhost:8080 and default values, so
they work for remote deployments and non-default workspaces.
- Around line 348-353: Update the “Confirm it worked” section near the API curl
command to distinguish persistence targets by workflow: for profile mode,
instruct users to verify entries in .nemo-optimizer/insights.yaml, and for API
mode, retain the API query command. Ensure the confirmation guidance reflects
the optimizer.yaml discovery behavior described earlier.
- Around line 348-353: Update the Insights API curl example under “Confirm it
worked” to include the Bearer Authorization header using NMP_ACCESS_TOKEN and
the --fail-with-body option, while preserving the existing URL and query
parameters.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 3e654248-c341-4a48-9d89-6cdf5fc63dc2

📥 Commits

Reviewing files that changed from the base of the PR and between af0e372 and 6ccf340.

⛔ Files ignored due to path filters (2)
  • docs/agents/_images/optimizer-experiment-comparison.png is excluded by !**/*.png
  • docs/agents/_images/optimizer-insight-detail.png is excluded by !**/*.png
📒 Files selected for processing (3)
  • docs/agents/insight-driven-optimization.mdx
  • docs/fern/gated-nav.yml
  • docs/fern/versions/latest.yml

Comment thread docs/agents/insight-driven-optimization.mdx
Comment thread docs/agents/insight-driven-optimization.mdx
@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor
Suite Lines Covered Line Rate Branch Rate
Unit Tests 29440/37448 78.6% 63.2%
Integration Tests 17379/36166 48.0% 20.6%

The two PNGs were the only binary files in the diff, and pre-commit's
stash rollback in the lint-merge-conflict step fails to re-apply a binary
patch over the working-tree churn left by the earlier regenerating lints.
The Studio URLs in the text still point readers at both views.

Signed-off-by: Jeremy Silva <jesilva@nvidia.com>
@jeremysilva1098 jeremysilva1098 changed the title add optimizer docs hidden from docs build docs: add optimizer docs hidden from docs build Jul 31, 2026
@github-actions github-actions Bot added the docs label Jul 31, 2026
@jeremysilva1098
jeremysilva1098 added this pull request to the merge queue Jul 31, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 31, 2026
@jeremysilva1098
jeremysilva1098 added this pull request to the merge queue Jul 31, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 31, 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.

2 participants