Skip to content
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.

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

Change all instances of taking argument "const Eigen::VectorXd &" to "const ConstVectorRef&" #27

Merged
merged 5 commits into from
Dec 18, 2024

Conversation

ManifoldFR
Copy link
Member

@ManifoldFR ManifoldFR commented Dec 18, 2024

This also the default constructor of the MPC class.

Merge this after #26

@ManifoldFR ManifoldFR requested a review from edantec December 18, 2024 14:50
@ManifoldFR ManifoldFR force-pushed the topic/more-eigen-refs-II branch 2 times, most recently from b9570e3 to 4a69f52 Compare December 18, 2024 15:12
@ManifoldFR ManifoldFR force-pushed the topic/more-eigen-refs-II branch from 4f60475 to ea7c00e Compare December 18, 2024 15:37
@ManifoldFR ManifoldFR force-pushed the topic/more-eigen-refs-II branch from ea7c00e to e3d858a Compare December 18, 2024 16:09
@ManifoldFR ManifoldFR changed the base branch from main to topic/ndcurves-fetchcontent December 18, 2024 16:28
@ManifoldFR ManifoldFR force-pushed the topic/more-eigen-refs-II branch from e3d858a to 8517cb9 Compare December 18, 2024 16:28
@ManifoldFR
Copy link
Member Author

ManifoldFR commented Dec 18, 2024

@ManifoldFR ManifoldFR changed the base branch from topic/ndcurves-fetchcontent to main December 18, 2024 16:31
@ManifoldFR ManifoldFR force-pushed the topic/more-eigen-refs-II branch from 8517cb9 to 32a7ec8 Compare December 18, 2024 16:31
@ManifoldFR ManifoldFR enabled auto-merge December 18, 2024 17:16
@ManifoldFR ManifoldFR merged commit 3e29a8d into main Dec 18, 2024
1 check passed
@ManifoldFR ManifoldFR deleted the topic/more-eigen-refs-II branch December 18, 2024 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants