10000 ENH: handle empty matrices in qr decomposition by convexset · Pull Request #11593 · numpy/numpy · GitHub
[go: up one dir, main page]
Skip to content

ENH: handle empty matrices in qr decomposition #11593

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.

Sign up for GitHub

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

Merged
merged 12 commits into from
Jul 31, 2018
Merged

ENH: handle empty matrices in qr decomposition #11593

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
36fcb0c
handle empty matrices in qr decomposition
convexset Jul 20, 2018
3fa693a
fixed offending quotes
convexset Jul 20, 2018
6dabc39
release notes updated
convexset Jul 20, 2018
448b409
revised to use existing array sizes; skipping calls to lapack in the …
convexset Jul 29, 2018
1624a80
setting things back where they were and calling LAPACK functions better
convexset Jul 29, 2018
e6fa7df
ensure lwork respects requirements of lapack methods (zgeqrf, dgeqrf,…
convexset Jul 29, 2018
7f22736
updated tests
convexset Jul 29, 2018
68d608e
simplified tests
convexset Jul 29, 2018
5809e9a
removed redundant test; expressed sub-test with pytest.mark.parametrize
convexset Jul 31, 2018
16d79e9
fixed dumb copy-and-paste error
convexset Jul 31, 2018
c4c6426
clarified terminology
convexset Jul 31, 2018
ff9063c
trimmed excess on line
convexset Jul 31, 2018
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
release notes updated
  • Loading branch information
@convexset
convexset committed Jul 20, 2018
commit 6dabc3937196efcd15d79e28e7e691dca3804ffe
6 changes: 6 additions & 0 deletions doc/release/1.16.0-notes.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,12 @@ Even when no elements needed to be drawn, ``np.random.randint`` and
distribution. This has been fixed so that e.g.
``np.random.choice([], 0) == np.array([], dtype=float64)``.

``linalg.qr`` now works with empty matrices
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Previously, a ``LinAlgError`` would be raised when empty matrix ("flat"
or "skinny") is passed in. This has been fixed so that outputs of
appropriate shapes are returned for the various modes.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if this and the similar note about lstsq for your related work in #11594 could be combined since they are both about handling empty matrices for linalg functions, but maybe that's a pain to coordinate between 2 separate PRs & could be done after I suppose.

Copy link
Contributor < 69D3 /span> Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Separated them because for lstsq, I was working in a strange way (site-packages). For qr it was a more straightforward "change and run tests".

ARM support updated
-------------------
Support for ARM CPUs has been updated to accommodate 32 and 64 bit targets,
Expand Down
0