Skip to content

Implement PTMRG#177

Open
VictorVanthilt wants to merge 4 commits intomasterfrom
vv-PTMRG
Open

Implement PTMRG#177
VictorVanthilt wants to merge 4 commits intomasterfrom
vv-PTMRG

Conversation

@VictorVanthilt
Copy link
Copy Markdown
Member

@VictorVanthilt VictorVanthilt commented Apr 25, 2026

This PR implements the PTMRG algorithm from https://arxiv.org/pdf/2411.12731
To do before merging:

  • Implement tests
  • Figure out how to get free energies from the data
  • Figure out which norms to take as data
  • Figure out how to normalize the h, v and C tensors
  • Check if the contraction ordering for the HOTRG projectors is optimal. (In HOTRG all legs have the same dimension, in PTMRG there is a big difference between the two tensors being merged)

this increases performance, and ensures there are no space mismatches
when using non-trivial spaces.
@VictorVanthilt
Copy link
Copy Markdown
Member Author

VictorVanthilt commented Apr 26, 2026

@sanderdemeyer You're calculating an eigenvalue decomposition of a map that isn't square on line 41 of c3vctm_honecomb.jl. Can you fix this?

@sanderdemeyer
Copy link
Copy Markdown
Contributor

@sanderdemeyer You're calculating an eigenvalue decomposition of a map that isn't square on line 41 of c3vctm_honecomb.jl. Can you fix this?

This problem should be resolved by #178.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 27, 2026

Codecov Report

❌ Patch coverage is 0% with 23 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/schemes/ptmrg.jl 0.00% 17 Missing ⚠️
src/utility/finalize.jl 0.00% 6 Missing ⚠️
Files with missing lines Coverage Δ
src/utility/finalize.jl 56.07% <0.00%> (-3.34%) ⬇️
src/schemes/ptmrg.jl 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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