search

Onepredict / onebone

An Open Source Signal Processing Library for Sensor Signals about vibration, current, etc.
MIT License
18 stars 3 forks source link

onebone.feature.snr 네이밍 관련 문의 #78

Open isingmodel opened 2 years ago

isingmodel commented 2 years ago

onebone.feature.snr (https://github.com/Onepredict/onebone/pull/59) 에 대한 문의입니다. 일반적으로 Signal to noise는 노이즈 대비 ROI 시그널의 신호 크기를 의미하는데, 어떤 성격의 시그널을 어떻게 분석하느냐에 따라 다른 방법이 쓰입니다. 이 측면에서 onebone.feature.snr에서는 너무 좁은 의미로 SNR을 구현한 것 아닌지 의구심이 듭니다. 함수 이름을 좀 더 직관적으로 바꾸면 어떨까요?

@seone-kim

Kangwhi-Kim commented 2 years ago

@seone-kim 함수를 만드실때 어떤 곳에 사용하는 목적으로 만드셨는지 혹은 제작 의도 반영하여 이름을 개선하는 방법도 좋다고 생각합니다. 혹시 어떻게 생각하시나요?

seone-kim commented 2 years ago

@isingmodel 안녕하세요, 우선 답변이 늦어 죄송합니다.

제가 구현한 SNR은 #59 에 적혀있는 것처럼 IEEE 2019에 출판된 논문에서 제시된 SNR을 구현한 방법입니다. 따라서 제가 작성한 SNR의 도메인은 전류 시그널, 로봇 도메인에 적합하다고 볼 수 있습니다. 분야에 따라 제가 작성한 함수가 적합하지 않을 수 있음에 동의합니다. 따라서 전류 도메인 등으로 feature 이름을 개선하는 것은 괜찮은 것 같습니다.

그 외 다른 이유나 아이디어가 있으시다면 의견주시길 바랍니다. 감사합니다.