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 hampel filter #85

Closed akangel0307 closed 2 years ago

akangel0307 commented 2 years ago

Summary

Write the summary about this PR.


  1. 노이즈가 있는 Signal noise_signal

  2. window_size =5 했을때 필터되는 signal data filtered_signal


Changes

Write changes in detail.

How to Test?

Write a way to test the changes.

Checklist

isingmodel commented 2 years ago
akangel0307 commented 2 years ago

넵 해당 내용은 지금 제가 하는 작업들이 끝나면 바로 def 형태로 넣겠습니다.

kyunghwan-onepredict commented 2 years ago
  • @kyunghwan-onepredict 저도 기범님 말씀에 동의하는데 현재 hampel-filter 를 경환님이 만든 filter.py 쪽에 def 로 포함시키는게 나을까요?? -> 이렇게 하시는 것이 좋을 것 같아요.

네 filter.py로 옮기시죠 ㅎ

kyunghwan-onepredict commented 2 years ago

@akangel0307 로직적인 부분은 거의 마무리 된것 같고, merge 전에 문서적인 부분에 대해서 주로 코멘트 남겼습니다. 얼마 안남았으니 힘내서 마무리 해봅시다 ㅎ

kyunghwan-onepredict commented 2 years ago

@all-contributors please add @akangel0307 for code

allcontributors[bot] commented 2 years ago

@kyunghwan-onepredict

I've put up a pull request to add @akangel0307! :tada: