Revert openblas linking changes from #451

[dime10]

PR #451 introduced linking against both OpenBLAS and LAPACKE+Accelerate on arm macOS, which breaks re-compilation of certain user programs (like Shor's). The cause of the shared library conflict is unknown.

For now, we revert these linking changes, which were originally introduced due to Lightning not finding its OpenBLAS dependency when invoked through Catalyst:

FAILED frontend/test/pytest/test_measurements_shots_results.py::TestExpval::test_hermitian[lightning.qubit]

RuntimeError: [pennylane_lightning/core/src/utils/SharedLibLoader.hpp][Line:60][Method:SharedLibLoader]: Error in PennyLane Lightning: dlopen(libscipy_openblas.dylib, 0x0001): tried: 'libscipy_openblas.dylib' (no such file), '$ORIGIN/../scipy_openblas32/lib/libscipy_openblas.dylib' (no such file)

The above test will fail as a result of this, which we plan to fix with a PR on the lightning side.

[sc-80803]

[github-actions]

Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md on your branch with:

• A one-to-two sentence description of the change. You may include a small working example for new features.
• A link back to this PR.
• Your name (or GitHub username) in the contributors section.

[multiphaseCFD]

LGTM! Thanks @dime10

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.82%. Comparing base (94b0bd6) to head (91ca827).
Report is 8 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1386      +/-   ##
==========================================
- Coverage   96.72%   95.82%   -0.91%     
==========================================
  Files          78       78              
  Lines        8439     8437       -2     
  Branches      873      873              
==========================================
- Hits         8163     8085      -78     
- Misses        222      299      +77     
+ Partials       54       53       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[erick-xanadu]

@dime10 there were some documentation changes to reflect support for finite shot measurements of hermitian observables. If this comes into effect, we should also revert those documentation changes.

[dime10]

@dime10 there were some documentation changes to reflect support for finite shot measurements of hermitian observables. If this comes into effect, we should also revert those documentation changes.

Well in theory they are supported simply by upgrading the version of lightning (which will be forced by next release anyway) 🤔 The original PR had no functional changes on the catalyst side besides the erroneous linking.

We could consider it a known issue on macos that the lapack libs may clash, and try to resolve it in a follow up?

[dime10]

We could consider it a known issue on macos that the lapack libs may clash, and try to resolve it in a follow up?

Having said that it might not be possible to merge this if the failure is reproducible in the CI.

[dime10]

Just waiting on the lightning wheels to become available and we should be able to merge this!