Skip to content

Comments

polish(nyz): fix redundant requires and vllm compatibility#36

Open
PaParaZz1 wants to merge 5 commits intomainfrom
dev-polish-vllm
Open

polish(nyz): fix redundant requires and vllm compatibility#36
PaParaZz1 wants to merge 5 commits intomainfrom
dev-polish-vllm

Conversation

@PaParaZz1
Copy link
Member

@PaParaZz1 PaParaZz1 commented Feb 3, 2026

TODO

  • remove requirements in setup.py
  • polish the library sequence requirements.txt
  • fix lowest transformers version (4.57.1)
  • fix hard requirements for vllm in code
  • add an optional choice for install vllm (vllm is not necessary for this repo), like pip install ".[vllm]"
  • add hint doc for install flash-attn issue (use whl or use docker)
  • sync pyproject.toml with requirements.txt
  • simplify gsm8k/geo3k shell, remove MLP_XXX
  • other necessary doc and comments

@PaParaZz1 PaParaZz1 added the polish Polish algorithms, tests or configs label Feb 3, 2026
@PaParaZz1 PaParaZz1 changed the title fix(nyz): fix redundant requires and vllm compatibility polish(nyz): fix redundant requires and vllm compatibility Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

polish Polish algorithms, tests or configs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants