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.

Sign up for GitHub

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

Jump to bottom

[OpenSora-HPCAI] Llava-Next Captioner #672

Merged
merged 1 commit into from
Sep 30, 2024
Merged

[OpenSora-HPCAI] Llava-Next Captioner #672

merged 1 commit into from
Sep 30, 2024

Conversation

zhtmike
Copy link
Collaborator

@zhtmike zhtmike commented Sep 26, 2024

  • Inference with single prompt and image
  • Inference with batch prompts and images

What does this PR do?

Fixes # (issue)

Adds # (feature)

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline?
  • Did you make sure to update the documentation with your changes? E.g. record bug fixes or new features in What's New. Here are the
    documentation guidelines
  • Did you build and run the code without any errors?
  • Did you report the running environment (NPU type/MS version) and performance in the doc? (better record it for data loading, model inference, or training tasks)
  • Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@xxx

SamitHuang
SamitHuang reviewed Sep 30, 2024
View reviewed changes
examples/opensora_hpcai/tools/caption/llava_next/tools/convert_llava.py
Copy link
Collaborator

Choose a reason for hiding this comment

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

can support load safetensor directly?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

sure, plan to do it in future

@SamitHuang SamitHuang added this pull request to the merge queue Sep 30, 2024
Merged via the queue into mindspore-lab:master with commit 65dadea Sep 30, 2024
3 checks passed
@zhtmike zhtmike deleted the llava_pr branch October 4, 2024 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SamitHuang SamitHuang SamitHuang approved these changes

@CaitinZhao CaitinZhao CaitinZhao approved these changes

@zhanghuiyao zhanghuiyao Awaiting requested review from zhanghuiyao zhanghuiyao is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zhtmike @SamitHuang @CaitinZhao