Qiskit / qiskit / 21942596356

88%

main: 88%

Build Type

push

github

Committed by web-flow

Commit Message

Fix ParameterExpression.sympify() for RPOW operations (#15642) (#15681)

* Add SciPyOptimizer (#6319)

* Add ScipyMinimizer

* fix test and lint

* use ScipyMinimizer to other scipy's optimizers

* support P_BFGS

* add doc strings and refactoring

* add max_evals_grouped

* add docstring for max_evals_grouped

* rename to SciPyOptimizer

* fix test

* Apply suggestions from code review



* refactor

* add releasenote



* Fix string formatting (#9540)

* Extend error message when target disjoint connectivity (#10733)

* Extend error message when target disjoint connectivity

* text in the exception

* Apply suggestions from code review



* changing the phrasing

---------



* Fix ParameterExpression.sympify() for RPOW operations

Fixes #15583

The sympify() method was not correctly handling reverse power
operations (RPOW). When converting ParameterExpression to sympy,
RPOW operations need to swap operands (rhs ** lhs instead of
lhs ** rhs) to match the Rust implementation.

This fix adds special handling for RPOW, RDIV, and RSUB operations
to correctly swap operands before applying the operation.

Contribution by Gittensor, see my contribution statistics at https://gittensor.io/miners/details?githubId=Eruis2579

* Add test and release note for RPOW sympify fix

- Add test_sympify_rpow_operand_order() to verify the fix for issue #15583
- Add release note documenting the bug fix

This addresses reviewer feedback to add tests and release notes.

* Add comprehensive tests for all reverse operations (RPOW, RDIV, RSUB)

This improves test coverage for the sympify fix by testing all three
reverse operation types that were added in the fix. This addresses
the Coveralls coverage report showing only 25% coverage.

* Fix POW operation to use RPOW when LHS is Parameter and RHS is expression

Fixes #15583

When a Parameter is the LHS and an expression (ParameterExpression) is the RHS
of a POW operation, we should use RPOW instead of POW. This ensures the repla... (continued)

Run Details

30 of 34 new or added lines in 2 files covered. (88.24%)

31 existing lines in 3 files now uncovered.

96681 of 109418 relevant lines covered (88.36%)

1203611.89 hits per line