Skip to content

a#31

Closed
mankai-chow wants to merge 10 commits into
ITensor:mainfrom
FuzzifiED:zheng_251113
Closed

a#31
mankai-chow wants to merge 10 commits into
ITensor:mainfrom
FuzzifiED:zheng_251113

Conversation

@mankai-chow

Copy link
Copy Markdown

a

@corbett5

corbett5 commented Nov 13, 2025

Copy link
Copy Markdown
Collaborator

Hi @mankai-chow, were you hoping to permit construction of operators with non-zero flux?

@mankai-chow

Copy link
Copy Markdown
Author

Hi @corbett5 , I was trying to merge a change to my own branch, but made a mistake and merged it to the main repo. You can neglect this pull request.

And yes, I was trying to permit constructions of operators with non-zero flux. I find that the package can do this correctly, but is forbidden.

@mankai-chow mankai-chow deleted the zheng_251113 branch November 13, 2025 18:15
@corbett5

Copy link
Copy Markdown
Collaborator

Understood. Yes it should be a simple change to check_os_for_errors to verify that each term has a consistent flux.

@corbett5

Copy link
Copy Markdown
Collaborator

@mankai-chow after merging #32 you can now construct MPOs with non-zero QN flux.

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