Skip to content

feat: pi agent server#1595

Merged
cmunley1 merged 11 commits into
mainfrom
cmunley/pi
Jun 26, 2026
Merged

feat: pi agent server#1595
cmunley1 merged 11 commits into
mainfrom
cmunley/pi

Conversation

@cmunley1

Copy link
Copy Markdown
Contributor

No description provided.

cmunley1 added 4 commits June 11, 2026 11:49
Signed-off-by: Christian Munley <cmunley@nvidia.com>
Signed-off-by: Christian Munley <cmunley@nvidia.com>
Signed-off-by: Christian Munley <cmunley@nvidia.com>
Signed-off-by: Christian Munley <cmunley@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Jun 14, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

cmunley1 added 2 commits June 13, 2026 18:30
Signed-off-by: Christian Munley <cmunley@nvidia.com>
Signed-off-by: Christian Munley <cmunley@nvidia.com>
@cmunley1 cmunley1 marked this pull request as ready for review June 14, 2026 01:31
@cwing-nvidia cwing-nvidia linked an issue Jun 23, 2026 that may be closed by this pull request
@ananthsub ananthsub requested a review from Glorf June 24, 2026 08:24
@anwithk anwithk moved this from Dev Todo to Review Todo in NeMo Gym 0.4.0 - July 1 Jun 25, 2026

@Glorf Glorf 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.

looks good overall, exact same set of comments as for #1594 (as these two are siblings)


if not shutil.which("pi"):
npm_bin_dir = subprocess.run(
[shutil.which("npm") or "npm", "bin", "-g"],

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.

same as opencode:

npm bin -g is a removed command.** npm 9 removed npm bin; bundled Node 22.15 ships npm 10 → returns empty, PATH fix no-ops, ends in the misleading "still not on PATH" error. Use npm prefix -g (+/bin) or npm root -g.


def model_post_init(self, __context: Any) -> None:
self.sem = Semaphore(self.config.concurrency)
ensure_pi(self.config.pi_version)

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.

same as opencode - would be good to retry

models:
- id: nvidia/qwen/qwen3-next-80b-a3b-instruct
reasoning: false
pi_version: null

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.

same as opencode - version pinning

cmunley1 added 3 commits June 25, 2026 14:41
Signed-off-by: Christian Munley <cmunley@nvidia.com>
Signed-off-by: Christian Munley <cmunley@nvidia.com>
@cmunley1 cmunley1 merged commit ba1474d into main Jun 26, 2026
9 checks passed
@cmunley1 cmunley1 deleted the cmunley/pi branch June 26, 2026 13:49
@github-project-automation github-project-automation Bot moved this from Review Todo to Done in NeMo Gym 0.4.0 - July 1 Jun 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Built-in harness integration: Pi

3 participants