-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Incorrect fermionic anticommutation relationships #102
Comments
I think both results are correct for bosons. I think the only thing |
I admit to being a bit rusty myself when it comes to With |
Hey thank you both for chipping in!
The above statement is true, however
this is not true: fermionic operator anticommute at different sites regardless of any symmetry of the state. That is Im not an expert on the package, but I believe this behaviour needs to be implemented to correctly characterise fermions (that is if it isn't already implemented and I am missing something!) |
@acroscarrillo no this asymmetry relation for fermions is not implemented. |
I see, I think that so long as a heads up is given in the documentation it should be okay.
|
Okay, I may have some ideas on how to implement the correct anticommutation relations, but this might be a breaking change, so RFC: Let us define the filling number states as follows: The ordering of the creation operators here is unimportant for bosons, but extremely important for fermions. Also for fermions we must have Thus, if we want to find the result of the creation operator, we can do the following: The result of the annihilation operator can be carried out in a similar way. The simplest way of somehow setting the particle statistics is adding a But do we really need that complication? Will file a PR ASAP if no objections. |
Thank you @acroscarrillo for reporting this and thank you @aryavorskiy for the fix. It is now merged (in #160 ) and a release will be made shortly. |
I'm fairly new to the package but it seems to me like the fermionic anticommutation relations
have been incorrectly implemented:
Since, if the code does what I think it should be doing, the last operation should be
but instead it is (wrongly) doing
as if they were bosons. Interestingly:
which is correct but looks like a fluke judging the above issue.
The text was updated successfully, but these errors were encountered: