Skip to content

Small Fixes#254

Open
AnanyaRaval wants to merge 6 commits into
mainfrom
bugfix/small-fixes
Open

Small Fixes#254
AnanyaRaval wants to merge 6 commits into
mainfrom
bugfix/small-fixes

Conversation

@AnanyaRaval
Copy link
Copy Markdown

@AnanyaRaval AnanyaRaval commented May 12, 2026

PR Type

Fix

Short Description

  1. Error while running venv.sh
$ bash venv.sh
:
error: unrecognized subcommand 'config'

Usage: uv [OPTIONS] <COMMAND>

For more information, try '--help'.
:
  1. Errors in example script in README.md
 from vec_inf.api import VecInfClient
ModuleNotFoundError: No module named 'vec_inf.api'
if status == ModelStatus.READY:
                 ^^^^^^^^^^^
NameError: name 'ModelStatus' is not defined
from vec_inf.client import wait_until_ready
ImportError: cannot import name 'wait_until_ready' from 'vec_inf.client'
  1. Fix docstring in vec_inf/client/api.py
 from vec_inf.api import VecInfClient
ModuleNotFoundError: No module named 'vec_inf.api'

Tests Added

  1. bash venv.sh
$ bash venv.sh
downloading uv 0.11.14 x86_64-unknown-linux-gnu
installing to /h/araval/.local/bin
  uv
  uvx
everything's installed!
Cache directory set to: /scratch/araval/uv_cache
Resolved 373 packages in 46ms
Checked 85 packages in 31ms
  1. Run script with changes and launch Qwen3 model:
$ uv run vec-inf status
┏━━━━━━━━┳━━━━━━━━━━━━┳━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━┓
┃ Job ID ┃ Model Name ┃ Status ┃ Base URL              ┃
┡━━━━━━━━╇━━━━━━━━━━━━╇━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━┩
│ 351427 │ Qwen3-8B   │ READY  │ http://bn083:44472/v1 │
│ 351428 │ Qwen3-8B   │ READY  │ http://bn083:64012/v1 │
│ 351429 │ Qwen3-8B   │ READY  │ http://bn083:23021/v1 │
└────────┴────────────┴────────┴───────────────────────┘

@AnanyaRaval AnanyaRaval force-pushed the bugfix/small-fixes branch from 3f562ff to ae4c24b Compare May 13, 2026 22:54
@AnanyaRaval AnanyaRaval marked this pull request as ready for review May 13, 2026 22:59
@AnanyaRaval AnanyaRaval requested review from MrGemy95 and amrit110 May 13, 2026 22:59
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.

1 participant