Skip to content

Fermionic fixes#174

Merged
lkdvos merged 14 commits into
masterfrom
fermionic_boundaries
Apr 18, 2025
Merged

Fermionic fixes#174
lkdvos merged 14 commits into
masterfrom
fermionic_boundaries

Conversation

@lkdvos

@lkdvos lkdvos commented Apr 11, 2025

Copy link
Copy Markdown
Member

Some missing parity matrices

@codecov

codecov Bot commented Apr 11, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 97.50000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/networks/local_sandwich.jl 85.71% 1 Missing ⚠️
Files with missing lines Coverage Δ
src/algorithms/contractions/vumps_contractions.jl 80.00% <100.00%> (+4.59%) ⬆️
src/algorithms/toolbox.jl 98.14% <100.00%> (+0.07%) ⬆️
src/utility/util.jl 54.20% <100.00%> (+3.20%) ⬆️
src/networks/local_sandwich.jl 76.31% <85.71%> (+2.12%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lkdvos
lkdvos force-pushed the fermionic_boundaries branch from 748ff6a to 70acd42 Compare April 11, 2025 13:49
Comment thread src/algorithms/toolbox.jl Outdated
Comment thread src/networks/local_sandwich.jl Outdated
@lkdvos
lkdvos force-pushed the fermionic_boundaries branch from d60ba6f to 28a2753 Compare April 15, 2025 12:03
@lkdvos
lkdvos enabled auto-merge (squash) April 15, 2025 13:21
@lkdvos
lkdvos requested a review from leburgel April 15, 2025 13:22

@leburgel leburgel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There's some accidental double twists due to the merge I think, and I would prefer the mpotensor converter to be a bit more in line with how we handle local sandwiches elsewhere, but looks great otherwise.

Comment thread src/algorithms/contractions/vumps_contractions.jl Outdated
Comment thread src/algorithms/contractions/vumps_contractions.jl Outdated
Comment thread src/algorithms/contractions/vumps_contractions.jl Outdated
Comment thread src/networks/tensors.jl Outdated
Comment thread src/utility/util.jl
@lkdvos
lkdvos force-pushed the fermionic_boundaries branch from 60c3e43 to 28a2753 Compare April 15, 2025 14:34
Comment thread src/networks/local_sandwich.jl

@leburgel leburgel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I added the mpotensor converter to the tests, good to go now for me.

@lkdvos
lkdvos merged commit 1d95d57 into master Apr 18, 2025
@lkdvos
lkdvos deleted the fermionic_boundaries branch April 18, 2025 12:43
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.

3 participants