search

mspass-team / mspass

Massive Parallel Analysis System for Seismologists
https://mspass.org
BSD 3-Clause "New" or "Revised" License
28 stars 11 forks source link

fix seismogram array bug in fileio #463

Closed Aristoeu closed 6 months ago

Aristoeu commented 9 months ago

461 fixed the index error in TimeSeries ensemble, I forgot the case for Seismogram ensemble.

The fix is the same as #461, adding a foff(0) for the dead seismogram , it will not be used but can prevent the index error.

codecov[bot] commented 9 months ago

Codecov Report

Merging #463 (58cb61b) into master (37e640a) will increase coverage by 0.00%. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #463   +/-   ##
=======================================
  Coverage   54.31%   54.31%           
=======================================
  Files         144      144           
  Lines       21956    21959    +3     
=======================================
+ Hits        11925    11928    +3     
  Misses      10031    10031
Files Coverage Δ
cxx/src/lib/io/fileio.cc 65.59% <100.00%> (+0.56%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Aristoeu commented 6 months ago

The error is fixed in #472