Skip to content

Commit f6e16b2

Browse files
authored
Merge pull request #3187 from stan-dev/fix/eigen-placeholder-all-to-ref-test
update to_ref test to use Eigen::all instead of Eigen::placeholders::all
2 parents cbb74b3 + ed6b54f commit f6e16b2

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

test/unit/math/prim/fun/to_ref_test.cpp

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,7 @@ TEST(MathMatrix, to_ref_matrix_exprs_tuple) {
2525
TEST(MathMatrix, to_ref_matrix_views_tuple) {
2626
Eigen::MatrixXd a = Eigen::MatrixXd::Random(3, 3);
2727
auto x = std::make_tuple(a.block(0, 0, 1, 1),
28-
a(Eigen::placeholders::all, std::vector{2, 1, 1}),
29-
a.array());
28+
a(Eigen::all, std::vector{2, 1, 1}), a.array());
3029
auto x_ref = stan::math::to_ref(x);
3130
using x_ref_t = decltype(x_ref);
3231
using stan::test::is_same_tuple_element_v;
@@ -46,11 +45,10 @@ TEST(MathMatrix, to_ref_matrix_views_exprs_tuple) {
4645
Eigen::MatrixXd a = Eigen::MatrixXd::Random(3, 3);
4746
auto x = std::make_tuple(
4847
a.block(0, 0, 1, 1),
49-
std::make_tuple(a.block(0, 0, 1, 1),
50-
a(Eigen::placeholders::all, std::vector{2, 1, 1}),
48+
std::make_tuple(a.block(0, 0, 1, 1), a(Eigen::all, std::vector{2, 1, 1}),
5149
a.array()),
5250
std::make_tuple(a * a, a, a.array() * 3),
53-
a(Eigen::placeholders::all, std::vector{2, 1, 1}), a.array() * a.array());
51+
a(Eigen::all, std::vector{2, 1, 1}), a.array() * a.array());
5452
auto x_ref = stan::math::to_ref(x);
5553
using x_ref_t = decltype(x_ref);
5654
using stan::test::is_same_tuple_element_v;
@@ -99,11 +97,11 @@ TEST(MathMatrix, to_ref_matrix_views_exprs_moves_tuple) {
9997
a.block(0, 0, 1, 1),
10098
std::forward_as_tuple(
10199
a.block(0, 0, 1, 1),
102-
a(Eigen::placeholders::all,
100+
a(Eigen::all,
103101
std::vector{Eigen::Index{2}, Eigen::Index{1}, Eigen::Index{1}}),
104102
a.array()),
105103
std::forward_as_tuple(a * a, a, a.array() * 3),
106-
a(Eigen::placeholders::all,
104+
a(Eigen::all,
107105
std::vector{Eigen::Index{2}, Eigen::Index{1}, Eigen::Index{1}}),
108106
a.array() * a.array()));
109107
using x_ref_t = decltype(x_ref);

0 commit comments

Comments
 (0)