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

Fix bounds parsing in Scipy optimizers and warn when unsupported #155

Merged
merged 46 commits into from
Apr 23, 2024
Merged
Changes from 1 commit
Commits
Show all changes
46 commits
Select commit Hold shift + click to select a range
bc4c98c
Avoid multiply defined bounds options in optimizer
edoaltamura Feb 13, 2024
85e16b6
Update change date
edoaltamura Feb 13, 2024
6604837
Add warnings for parsing unsupported bounds
edoaltamura Feb 14, 2024
77abbcb
Add algorithms and optimizer specific warnings
edoaltamura Feb 14, 2024
0163c53
Fix missed import
edoaltamura Feb 14, 2024
8167209
Merge branch 'main' into main
edoaltamura Feb 14, 2024
343ae7c
Apply style patches
edoaltamura Feb 15, 2024
d1db2a3
Add unit tests and introduce _bounds attribute
edoaltamura Feb 19, 2024
83df2c1
Formatting and final patches to tests
edoaltamura Feb 19, 2024
f4159ad
Merge branch 'qiskit-community:main' into main
edoaltamura Feb 20, 2024
7215fcc
Merge branch 'main' into main
woodsp-ibm Feb 22, 2024
28c5f67
Bounds parsing forbidden in __init__ and allowed in minimize()
edoaltamura Feb 23, 2024
a217786
Merge remote-tracking branch 'origin/main'
edoaltamura Feb 23, 2024
6d057ec
Add bounds type hinting
edoaltamura Feb 24, 2024
cdfc487
Change copyright in tests
edoaltamura Feb 24, 2024
7c0bf6b
Remove scikit-quant dependency
edoaltamura Feb 24, 2024
a4f45ee
Update qiskit_algorithms/optimizers/scipy_optimizer.py
edoaltamura Feb 24, 2024
18b0f77
Update qiskit_algorithms/optimizers/scipy_optimizer.py
edoaltamura Feb 27, 2024
140fce6
Update qiskit_algorithms/optimizers/scipy_optimizer.py
edoaltamura Feb 27, 2024
2fbd23a
Remove internal implementation checks in test_optimizers.py
edoaltamura Feb 27, 2024
d3eb8f2
Merge branch 'qiskit-community:main' into main
edoaltamura Feb 29, 2024
30c4b72
Avoid multiply defined bounds options in optimizer
edoaltamura Feb 13, 2024
06240b8
Update change date
edoaltamura Feb 13, 2024
be6dd75
Add warnings for parsing unsupported bounds
edoaltamura Feb 14, 2024
28d1493
Add algorithms and optimizer specific warnings
edoaltamura Feb 14, 2024
8446806
Fix missed import
edoaltamura Feb 14, 2024
f47d31a
Apply style patches
edoaltamura Feb 15, 2024
d7a0986
Add unit tests and introduce _bounds attribute
edoaltamura Feb 19, 2024
adfb264
Formatting and final patches to tests
edoaltamura Feb 19, 2024
cd7a34e
Bounds parsing forbidden in __init__ and allowed in minimize()
edoaltamura Feb 23, 2024
960cd7c
Add bounds type hinting
edoaltamura Feb 24, 2024
91ffeff
Change copyright in tests
edoaltamura Feb 24, 2024
ca0d277
Remove scikit-quant dependency
edoaltamura Feb 24, 2024
a7bbd21
Update qiskit_algorithms/optimizers/scipy_optimizer.py
edoaltamura Feb 24, 2024
3611957
Update qiskit_algorithms/optimizers/scipy_optimizer.py
edoaltamura Feb 27, 2024
08284c9
Update qiskit_algorithms/optimizers/scipy_optimizer.py
edoaltamura Feb 27, 2024
b9a32dc
Remove internal implementation checks in test_optimizers.py
edoaltamura Feb 27, 2024
7a5fc0c
Remove loose incompatible-bounds logic
edoaltamura Feb 29, 2024
f20fe98
Merge remote-tracking branch 'origin/main'
edoaltamura Feb 29, 2024
a467036
Remove loose incompatible-bounds logic
edoaltamura Feb 29, 2024
a8d52e0
Update qiskit_algorithms/optimizers/scipy_optimizer.py
edoaltamura Mar 7, 2024
0f602e7
Remove redundant `_bounds` attributes
edoaltamura Mar 7, 2024
1472f4f
Add release note
edoaltamura Mar 7, 2024
f4d76fa
Remove _bounds attr from tests, add `list[tuple[None, None]]` as poss…
edoaltamura Mar 13, 2024
997f38b
Remove _bounds attr from tests
edoaltamura Mar 13, 2024
602ab8c
Merge branch 'main' into main
woodsp-ibm Apr 23, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update qiskit_algorithms/optimizers/scipy_optimizer.py
Co-authored-by: Steve Wood <40241007+woodsp-ibm@users.noreply.github.com>
  • Loading branch information
edoaltamura and woodsp-ibm committed Feb 29, 2024
commit a7bbd21f25321bc87162b6aa17f0031a4401faa6
2 changes: 1 addition & 1 deletion qiskit_algorithms/optimizers/scipy_optimizer.py
Original file line number Diff line number Diff line change
@@ -78,7 +78,7 @@ def __init__(
self._max_evals_grouped = max_evals_grouped
self._kwargs = kwargs

# Initialise bounds and re-allocate if definition in options or kwargs
# Initialize bounds and re-allocate if definition in options or kwargs
self._bounds: list[tuple[float, float]] | None = None

if "bounds" in self._kwargs: