Skip to content

Time-evolution MPO refactoring#422

Merged
lkdvos merged 8 commits intomainfrom
time-mpo
May 3, 2026
Merged

Time-evolution MPO refactoring#422
lkdvos merged 8 commits intomainfrom
time-mpo

Conversation

@lkdvos
Copy link
Copy Markdown
Member

@lkdvos lkdvos commented May 3, 2026

In an attempt to reduce TTFX I refactored some of the code for the time-evolution MPO constructions, mostly just introducing a couple more function barriers.
Especially for the TaylorCluster implementation, the Val(N) dispatch should really help inference out quite a lot.
Additionally, this does actually affect runtime beyond TTFX, so overall this should be a good improvement. 🤞

@codecov
Copy link
Copy Markdown

codecov Bot commented May 3, 2026

Codecov Report

❌ Patch coverage is 97.32143% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/algorithms/timestep/taylorcluster.jl 96.15% 3 Missing ⚠️
Files with missing lines Coverage Δ
src/algorithms/timestep/wii.jl 100.00% <100.00%> (+1.85%) ⬆️
src/algorithms/timestep/taylorcluster.jl 98.08% <96.15%> (-1.92%) ⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lkdvos lkdvos merged commit 54389de into main May 3, 2026
29 of 30 checks passed
@lkdvos lkdvos deleted the time-mpo branch May 3, 2026 16:09
@VictorVanthilt
Copy link
Copy Markdown
Member

I'm curious, did this significantly improve TTFX?

@lkdvos
Copy link
Copy Markdown
Member Author

lkdvos commented May 4, 2026

Less than I was hoping :( it did actually speed up the code for small hamiltonians by quite a bit, since all of the bookkeeping stuff and indexing stuff was using non-inferred CartesianIndex{N} where {N}, so at least there is that

@lkdvos lkdvos mentioned this pull request May 4, 2026
lkdvos referenced this pull request May 5, 2026
* Update changelog for v0.13.11

* Bump version to v0.13.11

* manual changelog change

* Update CITATION.cff for v0.13.11

* attempt to precompile before tests
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