[refactor] scipy.linalg.misc is depracated: refactor imports#3423
[refactor] scipy.linalg.misc is depracated: refactor imports#3423tepals wants to merge 1 commit intodelmic:masterfrom
Conversation
fixes warning: DeprecationWarning: Please use `LinAlgError` from the `scipy.linalg` namespace, the `scipy.linalg.misc` namespace is deprecated
There was a problem hiding this comment.
Pull request overview
Refactors SciPy imports to eliminate the scipy.linalg.misc deprecation warning by importing LinAlgError directly from the supported scipy.linalg namespace.
Changes:
- Update
LinAlgErrorimport in core linear algebra utility module. - Update
LinAlgErrorimport in the corresponding unit test module.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
src/odemis/util/linalg.py |
Switches LinAlgError import to scipy.linalg to avoid deprecated scipy.linalg.misc. |
src/odemis/util/test/linalg_test.py |
Updates test import to match the new LinAlgError import path. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe pull request updates the import source for 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
fixes warning: DeprecationWarning: Please use
LinAlgErrorfrom thescipy.linalgnamespace, thescipy.linalg.miscnamespace is deprecated