adjtomo / pysep

Seismogram Extraction and Processing: Seismic data retrieval and record sections
https://pysep.readthedocs.io
MIT License
29 stars 15 forks source link

ZeroDivisionError while normalizing seismogram amplitudes during record section plotting #131

Closed aakash10gupta closed 6 months ago

aakash10gupta commented 8 months ago

I came across this special scenario where the user chooses a record section plotting time window where all seismograms have 0 amplitudes, and any kind of normalization leads to a ZeroDivisionError. Another potential scenario for this error is when only one or a few of the seismograms in the record section have all 0 values in the window, and are normalized with their own amplitudes. I think the expected behavior in both cases is to plot the seismograms as they are.

bch0w commented 8 months ago

Hi @aakash10gupta, I tried to fix this in #137, can you let me know if you're still experiencing this behavior on the latest devel branch?

aakash10gupta commented 6 months ago

This seems fixed now!