feat: Add moving average - Githubissues

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 moving average #30

Closed kbpark-onepredict closed 2 years ago

kbpark-onepredict commented 2 years ago

Summary

Write the summary about this PR.

Moving Average 구현 완료하였습니다.

Changes

Write changes in detail.

window size는 필수 인자로서 평균 계산을 할 범위를 나타냅니다.
pad는 input과 같은 길이를 원하는 경우에 사용할 수 있게 옵션으로 두었습니다. 맨 처음 값을 윈도우 크기 -1 만큼 padding 하는 방식으로 구현하였습니다.
weights는 weighted ma를 수행할 때 사용할 가중치입니다. 옵션으로 두었고 설정하지 않을 경우, 일반적인 ma를 수행합니다.
default 옵션, pad 사용, weights를 사용하였을 때의 결과를 함께 첨부합니다.

How to Test?

Write a way to test the changes.

1d 결과, 2d 결과, 3차원 이상에서의 에러 확인하였습니다.

Checklist

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

kbpark-onepredict commented 2 years ago

테스트 중 int -> float 변환 과정, 4/3 != 1.3333333....와 같은 작은 오차가 발생하여 np.equal 대신 np.isclose를 사용하였습니다.

kyunghwan-onepredict commented 2 years ago

PR 이름도 commit message 처럼 [type]: [Subject] 형태로 변경해주시면 감사하겠습니다. ex. feat: Add moving average algorithm

kbpark-onepredict commented 2 years ago

PR 이름도 commit message 처럼 [type]: [Subject] 형태로 변경해주시면 감사하겠습니다. ex. feat: Add moving average algorithm

반영했습니다.

kyunghwan-onepredict commented 2 years ago

@all-contributors please add @kbpark-onepredict code

allcontributors[bot] commented 2 years ago

@kyunghwan-onepredict

I've put up a pull request to add @kbpark-onepredict! :tada:

kyunghwan-onepredict commented 2 years ago

@kbpark-onepredict contributor 추가하는 commit 하였습니다. 👍