Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Model] LLaVA model refactor #4910

Merged
merged 2 commits into from
May 20, 2024
Merged

Conversation

DarkLight1337
Copy link
Collaborator

@DarkLight1337 DarkLight1337 commented May 20, 2024

Apply the model refactoring part of #4197 (shown here to pass CI) to make it easier to pinpoint why the CI fails for that PR.

@DarkLight1337 DarkLight1337 marked this pull request as draft May 20, 2024 02:50
@DarkLight1337 DarkLight1337 changed the title [Draft][Model] Test refactored LLaVA model [Draft][Model] LLaVA model refactor May 20, 2024
@DarkLight1337 DarkLight1337 changed the title [Draft][Model] LLaVA model refactor [Model] LLaVA model refactor May 20, 2024
@DarkLight1337 DarkLight1337 marked this pull request as ready for review May 20, 2024 03:40
Copy link
Collaborator

@ywang96 ywang96 left a comment

Choose a reason for hiding this comment

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

Overall LGTM - Left some comments!

vllm/model_executor/models/llava.py Outdated Show resolved Hide resolved
vllm/model_executor/models/llava.py Outdated Show resolved Hide resolved
vllm/model_executor/models/llava.py Outdated Show resolved Hide resolved
@DarkLight1337
Copy link
Collaborator Author

Overall LGTM - Left some comments!

I have edited the code accordingly.

@ywang96 ywang96 enabled auto-merge (squash) May 20, 2024 07:33
@ywang96 ywang96 merged commit 6287537 into vllm-project:main May 20, 2024
61 checks passed
@DarkLight1337 DarkLight1337 deleted the llava-refactor-1 branch May 20, 2024 08:12
dtrifiro pushed a commit to dtrifiro/vllm that referenced this pull request May 21, 2024
tybalex pushed a commit to tybalex/vllm-function-call that referenced this pull request May 25, 2024
mawong-amd pushed a commit to ROCm/vllm that referenced this pull request Jun 3, 2024
triple-Mu pushed a commit to CC-LLM/vllm that referenced this pull request Jun 5, 2024
robertgshaw2-neuralmagic pushed a commit to neuralmagic/nm-vllm that referenced this pull request Jun 8, 2024
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.

None yet

2 participants