Skip to content

Conversation

@andompesta
Copy link

What does this PR do?

Type of change: ?

Bug fix: default svd solver does not work for ill-conditioned matrixes. Switch to gesvd solver as it is more accurate (as per documentation) as does not fails for ill-conditioned matrixes

Overview: ?

Usage

# Add a code snippet demonstrating how to use this

Testing

Before your PR is "Ready for review"

  • Make sure you read and follow Contributor guidelines and your commits are signed.
  • Is this change backward compatible?: Yes
  • Did you write any new necessary tests?: No. as we might not want to test all matrix shapes
  • Did you add or update any necessary documentation?: No need to change the documentation
  • Did you update Changelog?: No

Additional Information

@andompesta andompesta requested a review from a team as a code owner December 8, 2025 10:56
@andompesta andompesta requested a review from jingyu-ml December 8, 2025 10:56
@copy-pr-bot
Copy link

copy-pr-bot bot commented Dec 8, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Signed-off-by: scavallari <[email protected]>
@andompesta andompesta force-pushed the scavallari_svd-quant_opt branch from 95f9b7e to c513c47 Compare December 8, 2025 10:58
@jingyu-ml
Copy link
Contributor

LGTM. As a reference, could you point me to the documentation?

@jingyu-ml jingyu-ml enabled auto-merge (squash) December 9, 2025 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants