Skip to content

Rewrite svd to avoid empty blocks#191

Merged
lkdvos merged 1 commit intomasterfrom
issue185
May 8, 2025
Merged

Rewrite svd to avoid empty blocks#191
lkdvos merged 1 commit intomasterfrom
issue185

Conversation

@lkdvos
Copy link
Member

@lkdvos lkdvos commented May 7, 2025

This avoids looping over blocks that will end up being truncated completely, ie that would get howmany = 0.

This probably fixes #185, although I don't have an explicit case to check this on.

@codecov
Copy link

codecov bot commented May 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ
src/utility/svd.jl 87.05% <100.00%> (+0.11%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lkdvos lkdvos requested a review from pbrehmer May 7, 2025 17:33
@Confusio
Copy link

Confusio commented May 7, 2025

This avoids looping over blocks that will end up being truncated completely, ie that would get howmany = 0.

This probably fixes #185, although I don't have an explicit case to check this on.

Thanks for the fixing. This will come out if we set svd_alg=SVDAdjoint(; fwd_alg=IterSVD()) in the boundary_alg in the following example:

boundary_alg = (; tol=1e-8, alg=:simultaneous, trscheme=(; alg=:fixedspace))

Copy link
Collaborator

@pbrehmer pbrehmer left a comment

Choose a reason for hiding this comment

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

Thanks for catching and fixing that!

@lkdvos lkdvos merged commit 7cfae1a into master May 8, 2025
39 checks passed
@lkdvos lkdvos deleted the issue185 branch May 8, 2025 00:20
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.

Stack on an empty collection for iterative svd

3 participants