Skip to content

feat: add allow_no_gpu option to skip GPU requirement during ingest#1668

Open
tomer-levin-nv wants to merge 8 commits intoNVIDIA:mainfrom
tomer-levin-nv:feat/allow-no-gpu-ingest
Open

feat: add allow_no_gpu option to skip GPU requirement during ingest#1668
tomer-levin-nv wants to merge 8 commits intoNVIDIA:mainfrom
tomer-levin-nv:feat/allow-no-gpu-ingest

Conversation

@tomer-levin-nv
Copy link

@tomer-levin-nv tomer-levin-nv commented Mar 20, 2026

Adds an allow_no_gpu flag (default False) to IngestorCreateParams, BatchIngestor, and resolve_requested_plan. When True, skips the "No GPUs available" error and returns a CPU-only RequestedPlan with all GPU allocations set to 0.0.

Made-with: Cursor

Description

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • If adjusting docker-compose.yaml environment variables have you ensured those are mimicked in the Helm values.yaml file.

@tomer-levin-nv tomer-levin-nv requested a review from a team as a code owner March 20, 2026 10:20
@copy-pr-bot
Copy link

copy-pr-bot bot commented Mar 20, 2026

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.

@tomer-levin-nv tomer-levin-nv force-pushed the feat/allow-no-gpu-ingest branch 2 times, most recently from 1641ccc to 42eaec6 Compare March 20, 2026 19:23
Adds an allow_no_gpu flag (default False) to IngestorCreateParams,
BatchIngestor, and resolve_requested_plan. When True, skips the
"No GPUs available" error and returns a CPU-only RequestedPlan with
all GPU allocations set to 0.0.

Made-with: Cursor
@tomer-levin-nv tomer-levin-nv force-pushed the feat/allow-no-gpu-ingest branch from 42eaec6 to 18bb25a Compare March 20, 2026 19:26
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.

2 participants