Skip to content

Conversation

@VictorVanthilt
Copy link
Member

I relaxed deduce_pspaces to make the user able to define an operator on a lattice without having to fully specify an operator on each site using @mpoham.
The behaviour now is the following:

  • If you supply an operator on every site no behaviour is changed since before.
  • If you only supply @mpoham with operators acting upon some of the lattice sites it will now either:
    1. Assume all sites have the same local Hilbert space if all the supplied operators have the same local Hilbert space.
    2. Error if the supplied operators have a different local Hilbert space.

@codecov
Copy link

codecov bot commented Apr 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ
src/operators/mpoham.jl 94.23% <100.00%> (+2.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.

@VictorVanthilt VictorVanthilt requested a review from lkdvos April 14, 2025 13:08
Copy link
Member

@lkdvos lkdvos left a comment

Choose a reason for hiding this comment

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

Thanks! Will merge once tests pass.

@VictorVanthilt VictorVanthilt merged commit 5879c1b into master Apr 14, 2025
9 checks passed
@lkdvos lkdvos deleted the vv-PSpacesRelaxation branch April 14, 2025 14:29
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