Skip to content

Add complex number support to linalg.qr - #548

Merged
rgommers merged 2 commits into
mainfrom
cmplx-linalg-qr
Dec 13, 2022
Merged

Add complex number support to linalg.qr#548
rgommers merged 2 commits into
mainfrom
cmplx-linalg-qr

Conversation

@kgryte

Copy link
Copy Markdown
Contributor

This PR

  • adds complex number support to linalg.qr. The main substantive change is updating the input and output data types.
  • adds an extended description describing the operation.
  • makes explicit that, for matrix stacks, the QR decomposition must be computed for each matrix in the stack.

@kgrytekgryte added API change Changes to existing functions or objects in the API. topic: Complex Data Types Complex number data types. topic: Linear Algebra Linear algebra. labels Dec 5, 2022
@kgrytekgryte added this to the v2022 milestone Dec 5, 2022

@rgommersrgommers left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Added content LGTM, thanks @kgryte

.. note::
Whether an array library explicitly checks whether an input array is a full column rank matrix (or a stack of full column rank matrices) is implementation-defined.

.. warning::

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This could be a note instead of a warning, in order to look less scary. That's a minor thing though, and perhaps a question of taste. So I'll leave it alone for now, just noting it in case someone revisits this PR in the future.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Followed PyTorch in using a warning, instead of a note.

@rgommers
rgommers merged commit 61694d0 into mainDec 13, 2022
@rgommers
rgommers deleted the cmplx-linalg-qr branch December 13, 2022 21:44
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API changeChanges to existing functions or objects in the API.topic: Complex Data TypesComplex number data types.topic: Linear AlgebraLinear algebra.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@kgryte@rgommers