Skip to content

[MRG] 🎨 add cusor rules and fix some small issues - #316

Merged
HuaizhengZhang merged 5 commits into
mainfrom
hz/vllm
Jul 15, 2025
Merged

[MRG] 🎨 add cusor rules and fix some small issues#316
HuaizhengZhang merged 5 commits into
mainfrom
hz/vllm

Conversation

@HuaizhengZhang

@HuaizhengZhangHuaizhengZhang commented Jul 14, 2025

Copy link
Copy Markdown
Contributor

Closes#313

  • add cursor project rules

Before submitting this PR, please make sure you have:

  • confirmed all checks still pass OR confirm CI build passes.
  • verified that any code or assets from external sources are properly credited in comments and/or in
    the credit file.

@HuaizhengZhangHuaizhengZhang self-assigned this Jul 14, 2025
@dosubotdosubotBot added size:L This PR changes 100-499 lines, ignoring generated files. enhancement New feature or request labels Jul 14, 2025
@HuaizhengZhangHuaizhengZhang changed the title [WIP] 🎨 refactor vllm engine support[MRG] 🎨 add cusor rules and fix some small issuesJul 15, 2025
@HuaizhengZhang

Copy link
Copy Markdown
ContributorAuthor

@syangx38 I have fixed the issue. No worry at all

@HuaizhengZhang

Copy link
Copy Markdown
ContributorAuthor

@YuanmingLeee no need to consider anything related to windows.

lets make our doc and code small and clean

@YuanmingLeee

Copy link
Copy Markdown
Contributor

@YuanmingLeee no need to consider anything related to windows.

lets make our doc and code small and clean

OK, since I dev on Windows, so note down the process

@YuanmingLeeeYuanmingLeee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@HuaizhengZhang
HuaizhengZhang merged commit 292d7fb into mainJul 15, 2025
3 checks passed

@huangyz0918huangyz0918 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@HuaizhengZhang In order to keep the mle-agent lite, we don't add any LLM model related dependencies into our lib's requirements.txt

Comment threadpyproject.toml
"google-api-python-client~=2.143.0",
"google-auth-httplib2~=0.2.0",
"google-auth-oauthlib~=1.2.1",
"google-genai~=1.25.0",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please remove the LLM related dependency

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementNew feature or requestsize:LThis PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Running mle new got google-genai dependency error

3 participants

@HuaizhengZhang@YuanmingLeee@huangyz0918