Skip to content

fix omni aligner #4117

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

Merged
merged 3 commits into from
May 7, 2025
Merged

fix omni aligner #4117

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3dcbad7
fix omni aligner
Jintao-Huang May 7, 2025
99f6f34
lint pass
Jintao-Huang May 7, 2025
747c119
update
Jintao-Huang May 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
lint pass
  • Loading branch information
@Jintao-Huang
Jintao-Huang committed May 7, 2025
commit 99f6f3419331209f8a69a46835b1589e0a4b3914
4 changes: 2 additions & 2 deletions swift/llm/train/tuner.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
from packaging import version
from transformers import TrainingArguments

from swift.llm import TrainArguments, get_model_arch, deep_getattr
from swift.llm import TrainArguments, deep_getattr, get_model_arch
from swift.plugin import Tuner, extra_tuners
from swift.tuners import Swift
from swift.utils import (activate_parameters, find_all_linears, find_embedding, find_norm, freeze_parameters,
Expand Down Expand Up @@ -88,7 +88,7 @@ def get_multimodal_target_regex(
if rejected_modules:
rejected_pattern = rf'(?!({"|".join(rejected_modules)}))'
res.append(rf'{rejected_pattern}{module}{target_pattern}')

return rf'^({"|".join(res)})$'


Expand Down
Loading