Onepredict / onebone

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

feat: Change minimum version 3.6 -> 3.8 #109

Closed kyunghwan-onepredict closed 2 years ago

kyunghwan-onepredict 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

codecov-commenter commented 2 years ago

Codecov Report

Merging #109 (b5a4b9c) into main (70a5a33) will increase coverage by 0.48%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #109      +/-   ##
==========================================
+ Coverage   81.07%   81.56%   +0.48%     
==========================================
  Files          24       24              
  Lines         560      564       +4     
==========================================
+ Hits          454      460       +6     
+ Misses        106      104       -2     
Flag Coverage Δ
unittests 81.56% <100.00%> (+0.48%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
onebone/feature/time.py 73.91% <100.00%> (ø)
onebone/signal/smoothing.py 79.68% <0.00%> (+4.68%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 70a5a33...b5a4b9c. Read the comment docs.

isingmodel commented 2 years ago

minimum 3.8은 너무 높지 않을까요?

kyunghwan-onepredict commented 2 years ago

minimum 3.8은 너무 높지 않을까요?

1.python 3.6은 공식적으로 지원이 종료되었습니다. Reference

  1. python 3.7부터 허용하려고 dependency들을 확인해보았습니다. 그런데 pandas, scipy 등 분석 알고리즘들의 최근 stable 버전이 python 3.8을 기준으로 작성되어 있는 것을 확인하였습니다. Reference: pandas, scipy
  2. 몇몇 라이브러리들의 버전을 3.7 기준으로 설정해보았지만, poetry에서 dependency를 제대로 풀지 못해 무한 로딩이 걸립니다. 아마 stable이 아닌 dependency 들을 잘 해소하지 못하는 문제가 있는것 같습니다.

이런 이유로 minimum 3.8로 설정하게 되었습니다. poetry dependency 문제가 python 3.7에서 해소된다면 3.7까지 허용할 생각이 있으나 현재 시점에서는 최소 버전을 3.8로 결정하였습니다.

혹시 @isingmodel 님께서도 해당 문제를 해결하는 방법을 찾게되면 알려주시기 바랍니다.

isingmodel commented 2 years ago

@kyunghwan-onepredict 벌써 3.6 지원 종료군요.. 말씀하신 포인트들에 동의합니다.

Kangwhi-Kim commented 2 years ago

기범님 LGTM

kbpark-onepredict commented 2 years ago

Good!