feat: Add the function for extracting the frequency features

Onepredict / onebone

An Open Source Signal Processing Library for Sensor Signals about vibration, current, etc.

MIT License

18 stars 3 forks source link

feat: Add the function for extracting the frequency features #13

Closed Kangwhi-Kim closed 2 years ago

Kangwhi-Kim commented 2 years ago

Summary

Write the summary about this PR.

Add the function for extracting the frequency features
If you want to know the features, see the reference; https://www.intechopen.com/chapters/40123.

Changes

Write changes in detail.

The function includes the mnf(mean frequency), mdf(median frequency), vcf(variance center frequency).

How to Test?

Write a way to test the changes.

$ python tests/signal/test_frequency_feature.py

Checklist

[x] Do you pass all the tests?
[ ] Do you write the changes on document such as README?

Kangwhi-Kim commented 2 years ago

TO DO

branch conflict 없애기 docstring 추가하기

isingmodel commented 2 years ago

directory 구조 측면에서, opllib.signal 안에 feature extraction code가 들어가는 것이 맞을까요?

Kangwhi-Kim commented 2 years ago

feature name을 약어로 해도 되나요? (만약 이 약어 표현들이 일반적으로 쓰이는 것들이라면 상관 없을 것 같네요. )

각 frequcny feature들 모두 얻는 경우가 꽤 있을 것 같은데, computational cost 차원에서 매 feature을 얻을 때마다 fft를 수행하는 것이 비효율적일 것 같아요. 혹시 input을 time domain이 아니라 frequency domain으로 넣는 것 어떨지요.

1> 약어를 쓴 배경은 np.std를 벤치마킹한 것이고, mnf, mdf, vcf 들은 계산수식을 참고한 서적에서 사용하는 것을 보고 이를 그대로 가져온 표현입니다. docstrings에 잘 표기하면 괜찮을 것 같습니다! (참고 서적 링크: https://www.intechopen.com/chapters/40123)

2> 좋은 지적 감사합니다. 계산 cost 차원에서 비효율적인 부분이 있네요. x (time signal) -> amp(스펙트럼 진폭)로 반영하겠습니다.

Kangwhi-Kim commented 2 years ago

directory 구조 측면에서, opllib.signal 안에 feature extraction code가 들어가는 것이 맞을까요?

cc: @kyunghwan-onepredict 이 부분은 앞으로의 알고리즘들이 추가되면서 수정 가능성이 높다고 생각합니다. 혹시 생각하신 폴더구조가 있을까요? 저는 앞으로 시간 도메인 피쳐들도 추가될 것을 감안할 때 oplib.stats 정도로 떠올려 보았습니다.

isingmodel commented 2 years ago

directory 구조 측면에서, opllib.signal 안에 feature extraction code가 들어가는 것이 맞을까요?

cc: @kyunghwan-onepredict 이 부분은 앞으로의 알고리즘들이 추가되면서 수정 가능성이 높다고 생각합니다. 혹시 생각하신 폴더구조가 있을까요? 저는 앞으로 시간 도메인 피쳐들도 추가될 것을 감안할 때 oplib.stats 정도로 떠올려 보았습니다.

oplib.stats은 statistical property의 성격이 강해보이니 oplib.feature.frequency 같은 건 어떨까합니다.

kyunghwan-onepredict commented 2 years ago

directory 구조 측면에서, opllib.signal 안에 feature extraction code가 들어가는 것이 맞을까요? cc: @kyunghwan-onepredict 이 부분은 앞으로의 알고리즘들이 추가되면서 수정 가능성이 높다고 생각합니다. 혹시 생각하신 폴더구조가 있을까요? 저는 앞으로 시간 도메인 피쳐들도 추가될 것을 감안할 때 oplib.stats 정도로 떠올려 보았습니다.

@Kangwhi-Kim @isingmodel 의견 감사합니다. feature라는 폴더가 여러가지로 많이 쓰일 것 같아 feature.frequency로 변경하겠습니다.