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

Implementation of two bundle methods #357

Merged
merged 388 commits into from
Feb 28, 2024
Merged

Conversation

hajg-ijk
Copy link
Member

@hajg-ijk hajg-ijk commented Feb 23, 2024

Implement the convex bundle method and the proximal bundle method.

  • Check code coverage.
  • Carefully recheck the documentation.

@kellertuer kellertuer added the Ready-for-Review A label for pull requests that are feature-ready label Feb 27, 2024
@kellertuer
Copy link
Member

We do have code coverage, all docs checked, code unified to rest of Manopt – this is good to go to be merged tomorrow.

@kellertuer kellertuer merged commit 8de71e4 into master Feb 28, 2024
15 checks passed
@kellertuer kellertuer deleted the hajg-ijk/bundle-method branch February 28, 2024 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Ready-for-Review A label for pull requests that are feature-ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants