Warnings cleanup

[j-c-c]

clean up warnings listed in #1398

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.66%. Comparing base (83badf4) to head (8fbac93).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1401      +/-   ##
===========================================
+ Coverage    88.18%   90.66%   +2.47%     
===========================================
  Files          135      135              
  Lines        14927    14927              
===========================================
+ Hits         13164    13533     +369     
+ Misses        1763     1394     -369     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[garrettwrong]

This looks good thanks. I'll watch for the eigh update for this PR mentioned in the meeting, but if that doesn't work out, you can use this as a backup.

[j-c-c]

This looks good thanks. I'll watch for the eigh update for this PR mentioned in the meeting, but if that doesn't work out, you can use this as a backup.

Thanks. Switched to eigh. The output of shrink_covar using eigh was allclose to the original implementation for all unit test cases.

[garrettwrong]

Sorry, hit "Ready For Review" instead of "Submit Review".

[janden]

LGTM