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
Fix 1675 #1684
Fix 1675 #1684
Conversation
(once we have consistent dtypes we can replace this with warnings and have the user set it reasonably)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only this minor thing. I'm unsure if it makes sense,
Otherwise thank you, looks good.
(Eventually it would be nice to get some comments in this code... but ok)
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #1684 +/- ##
==========================================
+ Coverage 82.42% 82.56% +0.14%
==========================================
Files 293 293
Lines 18049 18049
Branches 3482 3482
==========================================
+ Hits 14876 14902 +26
+ Misses 2497 2472 -25
+ Partials 676 675 -1 ☔ View full report in Codecov by Sentry. |
Make the "mask" implementation of FermiOperator2ndJax work with number of batch axes different from 1. Fixes netket#1675 (comment)). Also do some more casts to enforce the dtype of the configurations in the different implementations of the aforementioned operator, until one day we can set it consistently in e.g. hilbert.
Make the "mask" implementation of FermiOperator2ndJax work with number of batch axes different from 1. Fixes netket#1675 (comment)). Also do some more casts to enforce the dtype of the configurations in the different implementations of the aforementioned operator, until one day we can set it consistently in e.g. hilbert.
fixes #1675 (comment)
The extra two implementations are not currently tested (I did locally).
I also put some more casts to enforce the dtype of the configurations (until one day we can set it consistently in e.g. hilbert)