Skip to content

Conversation

@EunjuYang
Copy link
Contributor

Dependency of the PR

Commits to be reviewed in this PR

[bugfix] fix weight_converter.py
  • function call was missed.
  • typo is fixed

Self evaluation:

  1. Build test: [X]Passed [ ]Failed [ ]Skipped
  2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Eunju Yang [email protected]

Summary

  • Typo bugs are fixed

Signed-off-by: Eunju Yang [email protected]

- function call was missed.
- typo is fixed

Signed-off-by: Eunju Yang <[email protected]>
Copy link
Contributor

@songgot songgot left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Member

@myungjoo myungjoo left a comment

Choose a reason for hiding this comment

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

LGTM.

Not related to this PR, but the variable device (in qwen3-30b-a3b/80 and qwen3-4b/72) is unused.

Copy link
Collaborator

@jijoongmoon jijoongmoon left a comment

Choose a reason for hiding this comment

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

LGTM

@jijoongmoon jijoongmoon merged commit a7a79a1 into nnstreamer:main Dec 5, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants