Skip to content

a#31

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

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

Conversation

@mankai-chow
Copy link

a

@corbett5
Copy link
Collaborator

corbett5 commented Nov 13, 2025

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

@mankai-chow
Copy link
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
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
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