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 #82

Closed akangel0307 closed 2 years ago

akangel0307 commented 2 years ago

Summary

Write the summary about this PR.

Changes

Write changes in detail.

How to Test?

Write a way to test the changes.

Checklist

isingmodel commented 2 years ago
isingmodel commented 2 years ago
akangel0307 commented 2 years ago
  • test code 필요해요
  • labels -> enhancement로 해주세요
  • 함수에 설명이 없네요.

말씀해주신 부분 수정하였습니다! 감사합니다.

akangel0307 commented 2 years ago
  • 함수 위치는 signal/hampel_filter를 새로 만드는 것보다 signal/filter에 포함시키는 것이 좋을 겉 같아요.

@kyunghwan-onepredict 저도 기범님 말씀에 동의하는데 현재 hampel-filter 를 경환님이 만든 filter.py 쪽에 def 로 포함시키는게 나을까요??