feat(dev): add local model serving tool - #212
Conversation
da1d319 to
a3a8702
Compare
3f68c14 to
b57974c
Compare
981c1dc to
f8bd142
Compare
|
/nvskills-ci |
2 similar comments
|
/nvskills-ci |
|
/nvskills-ci |
|
/nvskills-ci |
fb09707 to
1dd931e
Compare
|
/nvskills-ci |
|
Post-rebase A100 dogfood completed on HEAD
The analyzer reproduced the seven previously verified measurement false positives (1 biographies, 6 legal). No leak-metric changes are included in this hoster PR. |
8093109 to
08b7332
Compare
|
/nvskills-ci |
08b7332 to
9d9536b
Compare
|
/nvskills-ci |
|
Added pinned GPT-OSS 120B, GPT-OSS 20B, and Qwen3 30B A3B Instruct profiles on rebased HEAD |
|
Medium hoster simplification is now on
The implementation began as an Engineer Arc candidate. Parent review rejected its over-aggressive test reduction, restored lifecycle-focused behavioral coverage, removed a stray compiler branch, and reran all validation before push. |
|
/nvskills-ci |
|
Added the local-model deployment docs and checked-in GPU container image in commit 0034b8f. Validation: full suite 1,226 passed; make check passed; strict docs build passed; Docker image build passed; packaged CLI and vLLM Factory compatibility doctor passed inside the image. |
|
/nvskills-ci |
|
Modern Python boundary refactor is now on HEAD
A Terra implementation and independent-review cycle found and resolved plugin-spec duplication, secret ownership, positional chunk tuples, and stale delegation seams. Primary-agent review then caught the empty-source-revision exit regression and the remaining manual HTTP ownership flag before push. Validation on the exact signed commit chain:
No bundled skill files changed, so no new |
|
Semantic cleanup is now on HEAD
A Terra reviewed-engineering Arc performed implementation, independent review, bounded fix-up, and confirmation. Review initially blocked on the PID-marker fallback and stale shipped instructions; both were fixed, and confirmation accepted the exact final tree with no findings. Validation on that tree:
|
|
/nvskills-ci |
|
Follow-up council warnings addressed in abf8a7e:
Validation:
|
|
Post-fix A100 dogfood passed on HEAD abf8a7e.
Run artifacts are retained locally under /tmp/anonymizer-abf8a7e-full-run-U7pB7p. |
|
Documented the existing NVIDIA W&B PR-vs-main scorecard workflow in |
|
Published a provenance-correct full-dataset PR-vs-main pair to the existing W&B scorecard.
Both 31-record measurement files were sealed and strictly imported with matching scorecard identities. The candidate uses measurement schema v1 and current main uses v2, so renamed replacement/leak counters appear separately instead of being treated as identical metrics. The refreshed snapshot was verified to contain both run IDs. Managed services were stopped cleanly; final GPU usage was 0 MiB. |
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
6bff538 to
836a13f
Compare
|
/nvskills-ci |
2 similar comments
|
/nvskills-ci |
|
/nvskills-ci |
Greptile SummaryThe PR adds a source-tree tool for compiling, launching, probing, monitoring, and stopping local vLLM inference services from pinned profiles.
Confidence Score: 5/5The PR appears safe to merge because no blocking failure remains within the eligible follow-up scope. No blocking failure remains; the previously reported per-request pooling fan-out is now constrained by both a chunk admission limit and a bounded worker frontier. Important Files Changed
|
| model=detection.model, | ||
| plugin=plugin, | ||
| text=chunk.text, | ||
| labels=detection.labels, | ||
| threshold=detection.threshold, | ||
| flat_ner=detection.flat_ner, | ||
| ) | ||
| for chunk in chunks |
There was a problem hiding this comment.
Unbounded pooling request fan-out
When a client submits a large detector request with chunk_length=1, overlap=0, and non-empty labels, this asyncio.gather eagerly schedules one pooling operation per character, exhausting serving-process memory and pooling capacity and disrupting other inference requests. Bound the number of chunks and concurrent pooling calls per request.
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Summary
Scope and lifecycle
The tool owns one deployment domain: a managed local-process vLLM server.
task,model,vllm, andlocal.compile,launch,probe,status, andstop.The tool remains under
tools/and is not part of the distributable Anonymizer package. It does not attach to existing endpoints or manage remote compute.Validation
Validation on rebased HEAD
836a13f:make check: passed, including formatting, lint,ty, lock consistency, and SPDX checks.make test: 1,307 passed and 3 expected vLLM Factory skips.make docs-build: strict build passed.main.Runtime evidence from the patch-equivalent pre-rebase tree:
tools/inference_service.Dockerfileas a real GPU image. The packaged CLI exposed all lifecycle commands and the vLLM Factory compatibility doctor passed.docs/data/NVIDIA_synthetic_biographies.csvwith zero workflow failures.cleanup_complete=true; final A100 memory use was 0 MiB.Fresh NVSkills validation is requested after the rebase so generated skill artifacts and signatures bind to the new commit chain.
Fixes #252