Removed inner and outer product functions.

[vprusso]

Closes: #830

[codecov]

Codecov Report

Attention: Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 97.9%. Comparing base (b07993c) to head (c0df19a).
Report is 77 commits behind head on master.

Files with missing lines	Patch %	Lines
toqito/matrix_props/sk_norm.py	50.0%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@          Coverage Diff           @@
##           master    #833   +/-   ##
======================================
  Coverage    97.8%   97.9%           
======================================
  Files         168     166    -2     
  Lines        3259    3249   -10     
  Branches      800     798    -2     
======================================
- Hits         3189    3181    -8     
+ Misses         46      44    -2     
  Partials       24      24           

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

[purva-thakre]

@vprusso The portion for removing inner and outer product functions looks good to me.

I'll take some time going over the sk_norm portion. Looks like you are also using this PR to fix #242.

[vprusso]

@vprusso The portion for removing inner and outer product functions looks good to me.

I'll take some time going over the sk_norm portion. Looks like you are also using this PR to fix #242.

@purva-thakre Yep, I added a few additional tests for coverage here in this PR. Part of the reason for that was the covbot thought somehow that removing those two functions reduced the overall test coverage, so I needed to up the coverage a bit by adding those lines. Kind of annoying, but I suppose more coverage isn't a bad thing!

[purva-thakre]

LGTM! Apologies for the delay.