Fix upon review

Update python/src/ops.cpp

Co-authored-by: Awni Hannun <awni.hannun@gmail.com>
This commit is contained in:
Hyunsung Lee 2025-04-22 05:47:07 +09:00
parent 3787167fb2
commit 5e8712cf6f

View File

@ -5194,26 +5194,18 @@ void init_ops(nb::module_& m) {
[](const nb::args& shapes) {
if (shapes.size() == 0)
throw std::invalid_argument(
"broadcast_shapes expects a sequence of shapes");
std::vector<mx::Shape> shape_vec;
shape_vec.reserve(shapes.size());
"[broadcast_shapes] Must provide at least one shape.");
mx::Shape result;
for (size_t i = 0; i < shapes.size(); ++i) {
if (!nb::isinstance<mx::Shape>(shapes[i]) &&
!nb::isinstance<nb::tuple>(shapes[i]))
throw std::invalid_argument(
"broadcast_shapes expects a sequence of shapes (tuple or list of ints)");
shape_vec.push_back(nb::cast<mx::Shape>(shapes[i]));
"[broadcast_shapes] Expects a sequence of shapes (tuple or list of ints).");
result = mx::broadcast_shapes(result, nb::cast<mx::Shape>(shapes[i]));
}
mx::Shape result = shape_vec[0];
for (size_t i = 1; i < shape_vec.size(); ++i)
result = mx::broadcast_shapes(result, shape_vec[i]);
auto py_list = nb::cast(result);
return nb::tuple(py_list);
return nb::tuple(nb::cast(result));
},
nb::sig("def broadcast_shapes(*shapes: Sequence[int]) -> Tuple[int]"),
R"pbdoc(