Skip to content

Commit 4a45eca

Browse files
committed
fixing tests to old versions
Summary Signed-off-by: HDCharles <[email protected]>
1 parent 07c657e commit 4a45eca

File tree

2 files changed

+16
-19
lines changed

2 files changed

+16
-19
lines changed

tests/llmcompressor/pytorch/modifiers/logarithmic_equalization/test_pytorch.py

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -8,18 +8,15 @@
88

99
@pytest.mark.unit
1010
def test_log_equalization_mapping(state):
11-
# Use regex patterns with parent-scoped search
12-
# Searches for balance layers within the parent of smooth layer
13-
mappings = [(["re:^fc2$"], "re:.*block1\\.fc1$")]
11+
mappings = [(["seq.fc2"], "seq.block1.fc1")]
1412
modifier = LogarithmicEqualizationModifier(mappings=mappings)
1513

1614
modifier.ignore = []
1715
modifier.resolved_mappings_ = modifier._resolve_mappings(state.model)
1816

19-
assert len(modifier.resolved_mappings_) == 1
17+
assert len(modifier.resolved_mappings_) == len(mappings)
2018

2119
mapping = modifier.resolved_mappings_[0]
22-
assert mapping.smooth_name == "seq.block1.fc1"
20+
assert mapping.smooth_name == mappings[0][1]
2321
assert isinstance(mapping.smooth_layer, Linear)
24-
assert len(mapping.balance_layers) == 1
25-
assert isinstance(mapping.balance_layers[0], Linear)
22+
assert isinstance(mapping.balance_layers[0], Linear)

tests/llmcompressor/pytorch/modifiers/smoothquant/test_pytorch.py

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -4,23 +4,23 @@
44
from llmcompressor.modifiers.smoothquant import SmoothQuantModifier
55

66

7+
import pytest
8+
from torch.nn import Linear
9+
10+
from llmcompressor.modifiers.smoothquant import SmoothQuantModifier
11+
12+
713
@pytest.mark.unit
814
def test_smooth_quant_mapping(state):
9-
# Use regex patterns with parent-scoped search
10-
# ^fc1$ matches only direct child "fc1", not nested "block1.fc1"
11-
mappings = [(["re:^fc1$"], "re:.*fc2$")]
15+
mappings = [(["seq.fc1"], "seq.fc2")]
1216
modifier = SmoothQuantModifier(mappings=mappings)
1317

1418
modifier.ignore = []
1519
modifier.resolved_mappings_ = modifier._resolve_mappings(state.model)
1620

17-
# Should match seq.fc2 and block1.fc2 (both end with fc2)
18-
assert len(modifier.resolved_mappings_) == 2
21+
assert len(modifier.resolved_mappings_) == len(mappings)
1922

20-
# Verify seq.fc2 mapping - should find only seq.fc1 (direct child)
21-
seq_mapping = [
22-
m for m in modifier.resolved_mappings_ if m.smooth_name == "seq.fc2"
23-
][0]
24-
assert isinstance(seq_mapping.smooth_layer, Linear)
25-
assert len(seq_mapping.balance_layers) == 1
26-
assert isinstance(seq_mapping.balance_layers[0], Linear)
23+
mapping = modifier.resolved_mappings_[0]
24+
assert mapping.smooth_name == mappings[0][1]
25+
assert isinstance(mapping.smooth_layer, Linear)
26+
assert isinstance(mapping.balance_layers[0], Linear)

0 commit comments

Comments
 (0)