v0.11.0 #233
Merged
v0.11.0 #233
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Merge pull request #193 from PyLops/dev
doc: fix minor mistake in HuberCircular docstring
feat: added gfirst to ADMML2
feat: added grad of Moreau envelope to grad method of ProxOperator
fix: changed L01Ball into L10Ball
Documentation revamp
The current implementation of the L0 norm (__call__ of L0) counts the values above sigma, however this should be 0. This is now fixed and tests for the L0 operator are added.
bug: fix bug in L0 __call__ and add tests
minor: fix comment of ValueError in _PrecompositionOperator
bug: fixes #200
feat: add HalfSpace
First attempt to add support for initializing also the auxiliary variable in various solvers, and modified logic for internal initialization based on whether x0 or z0/y0 is provided.
- old: (x > self.lower) & (x < self.upper): this is an open set, which violates the assumption of the convex projection - fix (x >= self.lower) & (x <= self.upper): this is a closed convex set. also add a test for this logic.
feat[BREAKING]: initial guess of auxiliary variable
- move and change lines to be consistent with other tests
fix the Box indicator function
- DykstrasProjection - (Parallel) Dykstra's projection algorithm computing convex projection to the intersection of convex sets - DykstrasProjectionProx - The corresponding indicator function (prox) - DykstraLikeProximal - (Parallel) Dykstra-like proximal algorithm computing prox of the sum of convex functions Add tests in a separate test file. - test_dykstra.py
- minor fix in test_dykstras_projection() - change np to ncp with get_array_module() - improve docstring and comments in DykstraLikeProximal.py - refactoring DykstraLikeProximal.__call()__ for readability - refactoring DykstraLikeProximal._parallel_dykstra_like_proximal_algorithm() for reducing the cyclomatic complexity to pass the CI during PR
- to pass the CI during PR
- rename classes - DykstrasProjectionProj --> GenericIntersectionProj - DykstrasProjectionProx --> GenericIntersectionProx - DykstraLikeProximal --> Sum - separate core algorithms to _dykstra_core.py to reduce duplication - docstring improved - add examples to dykstra.py
feat: Add Dykstra's algorithms and tests.
ci: try to bring back macos into GA
Prepare for v0.11.0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.