Skip to content
This repository has been archived by the owner on Jan 19, 2023. It is now read-only.

Update README.md + fixed taking square roots to avoid NANs #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

brando90
Copy link

added

pip install -U git+https://github.com/google/svcca

to allow installations

@brando90
Copy link
Author

important change so code does not returns nans:

  import scipy
  # print('npy\'s sqrt')
  invsqrt_xx = scipy.linalg.sqrtm(inv_xx)
  invsqrt_yy = scipy.linalg.sqrtm(inv_yy)

instead of

  # print('their sqrt')
  # invsqrt_xx = positivedef_matrix_sqrt(inv_xx)
  # invsqrt_yy = positivedef_matrix_sqrt(inv_yy)

@brando90 brando90 changed the title Update README.md Update README.md + fixed taking square roots to avoid NANs Nov 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant