Skip to content

Commit d49696b

Browse files
committed
fixed error in tests
1 parent 33d971c commit d49696b

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

tests/test_gbt_single.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -216,13 +216,13 @@ def test_compress_cpu(self):
216216
new_tree_indices = np.zeros(n_compressed_trees)
217217
new_tree_indices[1:] = np.cumsum(n_leaves_per_tree[compressed_tree_indices])[:-1]
218218

219-
# Create zero correction matrix W with shape (n_leaves + 1, output_dim)
220-
# W is sized for original ensemble - corrections are applied before copying
221-
W = np.zeros((n_leaves + 1, self.out_dim), dtype=np.single)
219+
# Create zero correction matrix W_compressed with shape (n_compressed_leaves + 1, output_dim)
220+
# Row 0 is bias, remaining rows are for compressed leaves only
221+
W_compressed = np.zeros((n_compressed_leaves + 1, self.out_dim), dtype=np.single)
222222

223223
model.learner._cpp_model.compress(
224224
n_compressed_leaves, n_compressed_trees, compressed_leaf_indices,
225-
compressed_tree_indices, new_tree_indices.astype(np.int32), W)
225+
compressed_tree_indices, new_tree_indices.astype(np.int32), W_compressed)
226226

227227
compressed_y = model(X, tensor=False)
228228
self.assertTrue(np.allclose(compressed_y, y_pred_k),
@@ -295,13 +295,13 @@ def test_compress_gpu(self):
295295
new_tree_indices = np.zeros(n_compressed_trees)
296296
new_tree_indices[1:] = np.cumsum(n_leaves_per_tree[compressed_tree_indices])[:-1]
297297

298-
# Create zero correction matrix W with shape (n_leaves + 1, output_dim)
299-
# W is sized for original ensemble - corrections are applied before copying
300-
W = np.zeros((n_leaves + 1, self.out_dim), dtype=np.single)
298+
# Create zero correction matrix W_compressed with shape (n_compressed_leaves + 1, output_dim)
299+
# Row 0 is bias, remaining rows are for compressed leaves only
300+
W_compressed = np.zeros((n_compressed_leaves + 1, self.out_dim), dtype=np.single)
301301

302302
model.learner._cpp_model.compress(
303303
n_compressed_leaves, n_compressed_trees, compressed_leaf_indices,
304-
compressed_tree_indices, new_tree_indices.astype(np.int32), W)
304+
compressed_tree_indices, new_tree_indices.astype(np.int32), W_compressed)
305305

306306
compressed_y = model(X, tensor=False)
307307
self.assertTrue(np.allclose(compressed_y, y_pred_k),

0 commit comments

Comments
 (0)