Use eigh in bath op assertion

[piperfw]

#165

The constructor already asserts the coupling operator is Hermitian, so using eigh should be safe and not fail in the way described in the issue.

Never done a one-character Pull Request before. Ran the tests just to be sure!

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.12%. Comparing base (0015fc2) to head (4b5eb02).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #166   +/-   ##
=======================================
  Coverage   92.12%   92.12%           
=======================================
  Files          24       24           
  Lines        4572     4572           
=======================================
  Hits         4212     4212           
  Misses        360      360           

☔ 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.

[piperfw]

If we wanted to be more thorough, we could write some tests using an operator such as in the issue example

[gefux]

Because we are checking a few lines above that tmp_coupling_operator is hermitian np.linalg.eigh is indeed the better choice.
I'll add the test (following the issue #165) in a sec.

[gefux]

This closes issue #165.

Thank you, @piperfw, for this wold record single character PR. I am almost sorry I messed it up in the end with adding a test. :)

[piperfw]

I cling to that my request was single-character. What you did with that request is neither here nor there!