8000 Fix external casadi functions with sparse parameter definitions, remove unused memory, add tests by FreyJo · Pull Request #1162 · acados/acados · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix external casadi functions with sparse parameter definitions, remove unused memory, add tests #1162

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 17 commits into from
Jul 22, 2024
Merged

Fix external casadi functions with sparse parameter definitions, remove unused memory, add tests #1162

merged 17 commits into from
Jul 22, 2024

Conversation

FreyJo
Copy link
Member
@FreyJo FreyJo commented Jul 19, 2024 •
edited
Loading
  • fix: external_function_param_casadi class for the case when the parameters are not a dense vector. This is still not recommended
  • removed unused memory requirement of np in external_function_param_casadi.
  • added static functions d_cvt_ext_fun_arg_to_casadi, d_cvt_casadi_to_ext_fun_arg to avoid code duplication

Tests:

  • added examples/acados_python/tests/sparse_param_test.py
  • improved test of parametric OCP formulation

@FreyJo FreyJo changed the title Investigated sparse parameters Fix external casadi functions with sparse parameter definitions, remove unused memory, add tests Jul 21, 2024
@FreyJo FreyJo force-pushed the investigate_sparse_params branch 2 times, most recently from a6b89c4 to ed015a4 Compare July 21, 2024 10:24
@FreyJo FreyJo force-pushed the investigate_sparse_params branch from ed015a4 to 3f15094 Compare July 21, 2024 10:53
@FreyJo FreyJo force-pushed the investigate_sparse_params branch from 3f15094 to c580b5d Compare July 22, 2024 07:32
@sandmaennchen sandmaennchen merged commit d0af625 into acados:master Jul 22, 2024
10 checks passed
@sandmaennchen sandmaennchen deleted the investigate_sparse_params branch July 22, 2024 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sandmaennchen sandmaennchen sandmaennchen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@FreyJo @sandmaennchen
0