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
Catplot fix #2739
Catplot fix #2739
Conversation
eroell
commented
Nov 9, 2023
•
edited
edited
- Closes Plotting with using scanpy.pl gives attribute error. #2680
- Tests included or not required because: this is a longterm fix - failing of this functionality would be captured by existing tests
- Release notes not necessary because:
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #2739 +/- ##
==========================================
+ Coverage 72.92% 73.13% +0.20%
==========================================
Files 111 111
Lines 12134 12134
==========================================
+ Hits 8849 8874 +25
+ Misses 3285 3260 -25
|
To see tracebacks, you need to scroll up from the end past the enormous warnings list (I’m working on that), then you see the actual error E UserWarning: The problem size 5 minus the constraints size 0 is too small relative to the block size 4. Using a dense eigensolver instead of LOBPCG iterations.No output of the history of the iterations.
/opt/hostedtoolcache/Python/3.11.6/x64/lib/python3.11/site-packages/scipy/sparse/linalg/_eigen/lobpcg/lobpcg.py:493: UserWarning This test has stared to become flaky some time ago, needs some investigation to make this work reliably again. |
Yeah noted there is an issue with scipy... Not related to this the single-line update? |
Not related to your changes, no. You can see that e.g. the “PyTest anndata_dev” job succeeded |
thx! |
Co-authored-by: Eljas Roellin <65244425+eroell@users.noreply.github.com>