-
Notifications
You must be signed in to change notification settings - Fork 24
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
Physical #447
Physical #447
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #447 +/- ##
===========================================
+ Coverage 88.75% 88.80% +0.05%
===========================================
Files 100 100
Lines 6982 7024 +42
===========================================
+ Hits 6197 6238 +41
- Misses 785 786 +1
Continue to review full report in Codecov by Sentry.
|
- Addressing CodeFactor's issues. - Improving the code for is_CP and is_positive (for Channels and DMs respectively).
… ) as input for modes. - Added the missing test for DM.is_physical
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.
Looking good so far mostly just updates to the docs
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.
Super! 💯
I agree with Anthony's suggestions.
- Added .is_physocal for Kets. - Improved code slightly
- Adding self.ATOL in the utils of mrmustard and setting it to 1e-8. This tolerance level is used in checking the positivity conditions and CP conditions. - Improving the code when checking chonditions in the form all(...).
Context:
Physicality checks added.
Description of the Change:
One can check physicality of a DM
rho
, or a Channelch
usingrho.is_physical
and/orch.is_physical
.Moreover, one can check
ch.is_TP
,ch.is_CP
, andDM.is_positive
.Benefits:
Clear.
Possible Drawbacks:
None.
Related GitHub Issues:
None.