search

andreww / MSAT

MATLAB Seismic Anisotropy Toolkit
http://www1.gly.bris.ac.uk/MSAT/
Other
42 stars 29 forks source link

MS_Phasevels does not checkC #2

Closed andreww closed 12 years ago

andreww commented 12 years ago

We don't run the C (elasticity matrix) input argument to MS_Phasevels through MS_checkC. If invalid (e.g. non-positive definite) C is provided we can then return complex numbers for the anisotropy - which breaks e.g. MS_plot. We may want to make this check optional.

andreww commented 12 years ago

Fixed by commit b7d3c1c. Check is not optional (we cannot pass in an array of C's so the time taken here is trivial.