paudetseis / RfPy

Teleseismic receiver function calculation and post-processing
https://paudetseis.github.io/RfPy/
MIT License
40 stars 28 forks source link

Time indexing bug fix in wiggle plotting function #14

Closed jaredbryan881 closed 4 years ago

jaredbryan881 commented 4 years ago

Calling plotting.wiggle produces ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all() on line 89. and evaluates the truth value of the entire array object, which is not well-defined, so I changed it to a bitwise &, which makes boolean evaluations on the values of the time index arrays rather than the arrays themselves.

codecov-commenter commented 4 years ago

Codecov Report

Merging #14 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master     #14   +/-   ##
======================================
  Coverage    5.17%   5.17%           
======================================
  Files          10      10           
  Lines        2455    2455           
======================================
  Hits          127     127           
  Misses       2328    2328           
Impacted Files Coverage Δ
rfpy/plotting.py 5.04% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 4913104...626fb3d. Read the comment docs.

paudetseis commented 4 years ago

Thanks @jaredbryan881 - I will merge this immediately