Skip to content

Commit b1d3314

Browse files
authored
Merge pull request #397 from Zionshang/fix-issue-396
fix: use nu_max for du_tmp buffer allocation in tryLinearStep (#396)
2 parents 42ae195 + e7544bc commit b1d3314

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
1010
### Fixed
1111

1212
- Update finite difference helper to support explicit dynamics, replacing the obsolete implicit-only implementation. (https://github.com/Simple-Robotics/aligator/pull/392)
13+
- Fix `SolverProxDDP::tryLinearStep()` temporary control update buffer allocation to use `nu_max` instead of `ndx_max`. (https://github.com/Simple-Robotics/aligator/pull/397)
14+
1315

1416
## [0.18.0] - 2026-01-27
1517

include/aligator/solvers/proxddp/solver-proxddp.hxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ Scalar SolverProxDDPTpl<Scalar>::tryLinearStep(const Problem &problem,
129129
}
130130
ndx_max = std::max(ndx_max, workspace_.dxs.back().size());
131131
ArenaMatrix<VectorXs> dx_tmp{ndx_max, allocator_};
132-
ArenaMatrix<VectorXs> du_tmp{ndx_max, allocator_};
132+
ArenaMatrix<VectorXs> du_tmp{nu_max, allocator_};
133133

134134
for (size_t i = 0; i < nsteps; i++) {
135135
const StageModel &stage = *problem.stages_[i];

0 commit comments

Comments
 (0)