enable partially entangling ms gates

[splch]

adds a third parameter for the ms gate that defaults to 0.25 (fully entangling). can be changed to enable partial entanglement.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.06%. Comparing base (a7d2d30) to head (8094647).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #101      +/-   ##
==========================================
- Coverage   96.85%   96.06%   -0.80%     
==========================================
  Files           5        5              
  Lines         318      330      +12     
==========================================
+ Hits          308      317       +9     
- Misses         10       13       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[trbromley]

Thanks a lot @splch! I'll tag someone from the team to take a second look.

[splch]

thanks! i appreciate the suggestion too :)

[albi3ro]

Looks straightforward enough.

Mind adding a change log entry at:

https://github.com/PennyLaneAI/PennyLane-IonQ/blob/master/CHANGELOG.md

[trbromley]

Thanks @splch! Looks like it's nearly there, but missing a test that passes the theta parameter. Are you able to add that in?

[splch]

Thanks @splch! Looks like it's nearly there, but missing a test that passes the theta parameter. Are you able to add that in?

hi! thanks for the feedback :) is this what you were thinking of for theta testing? https://github.com/PennyLaneAI/PennyLane-IonQ/pull/101/files#diff-72107d6f4a703abaeb6a7cf709e0c99f7ed0b7c74f0b14b0cbc4f35c7c650e26R369-R389

[Alex-Preciado]

Thank you so much for this contribution @splch !! We have double checked and the Codecov failures seem unrelated to the changes made in this PR. Force merging now!! 🚀