return SVD from pinv(::SVD)

[stevengj]

As suggested by @andreasnoack in #1387 (comment), this changes the pinv(::SVD) function to instead return the SVD of the pseudo-inverse, rather than an explicit matrix, so that it can be applied stably.

(No backwards-compatibility issue since the pinv(::SVD) method was introduced in #1387.)

[stevengj]

Whoops, failing because *(::SVD, ::AbstractVector) is not defined. This should be fixed — otherwise returning an SVD object from pinv is useless.

Update: fixed.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.85%. Comparing base (b7fd696) to head (488bc09).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1398      +/-   ##
==========================================
- Coverage   93.87%   93.85%   -0.02%     
==========================================
  Files          34       34              
  Lines       15826    15829       +3     
==========================================
+ Hits        14856    14857       +1     
- Misses        970      972       +2     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[stevengj]

CI failure on Windows is an unrelated server glitch.