Skip to content

fix:rename VLLM_SPYRE ENV to SENDNN_INFERENCE#60

Open
bayo-ibm wants to merge 1 commit into
IBM:devfrom
bayo-ibm:dev2
Open

fix:rename VLLM_SPYRE ENV to SENDNN_INFERENCE#60
bayo-ibm wants to merge 1 commit into
IBM:devfrom
bayo-ibm:dev2

Conversation

@bayo-ibm
Copy link
Copy Markdown

@bayo-ibm bayo-ibm commented May 7, 2026

The PR seeks to replace the naming convention for environment variables from VLLM_SPYRE* to SENDNN_INFERENCE*. This aligns with the change in the naming convention in the new VLLM version (please see link)

Signed-off-by: Omobayode Fagbohungbe <omobayode.fagbohungbe@email.com>
@WarningRan
Copy link
Copy Markdown
Collaborator

Hi @bayo-ibm, thanks for the rename!

Could you paste a short summary of the Slack context into the PR description? It'd help future readers (and anyone doing git blame later) to have it on GitHub rather than only in Slack. The key points worth capturing:

  • Part of the broader vllm_spyresendnn_inference migration (Python package rename, VLLM_PLUGINS=spyresendnn_inference, plus the env vars in this PR).
  • VLLM_DT_* backend vars are intentionally unchanged.
  • Lands with the 3.5 EA1 / 2.0 swap.

One question: is this a hard cutover with 3.5 EA1, or should fmwork keep dual-name compatibility for a while? Our downstream log parsing / JSON post-processing needs to know whether to support both names.

Thanks!

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