obspy icon indicating copy to clipboard operation
obspy copied to clipboard

Problem with signal.freqattributes

Open megies opened this issue 12 years ago • 2 comments

Currently some routines (e.g. cfrequency) seem to not work with unwindowed data, i.e. a single time series and thus a 1D array as input data, although the docstrings suggest thiat this is supported. These routines fail due to a data.shape[1] which gives an IndexError (e.g. here). This could be prevented by making 1D inputs into a 2D array. However, this could lead to other problems as later on some checks of the dimensionality are used to control the routine (e.g. here). This needs some refactoring.

megies avatar Mar 20 '13 09:03 megies

My fix only touched one of the routines from signal.freqattributes. There are others that still need refactoring. I guess this issue should stay open.

amaggi avatar Oct 31 '13 14:10 amaggi

Oh, I see. OK.

megies avatar Oct 31 '13 14:10 megies