Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow latest versions of dependencies #181

Merged
merged 15 commits into from
Dec 30, 2021
Merged

Conversation

gcbeltramini
Copy link
Contributor

@gcbeltramini gcbeltramini commented Dec 28, 2021

Status

READY

Todo list

  • Tests added and passed

Background context

Most of the versions were added many months ago, and their breaking changes were mostly due to the drop of support to Python 2.7 and 3.5.

Description of the changes proposed in the pull request

  • Keep the minimum required version of the libs and allow their latest versions, except for shap, whose version is less than 1, so it can be considered experimental.
  • Allow greater versions of catboost, lightgbm, xgboost, shap, swifter (mostly due to deprecation of support to Python 3.5 and older versions). Libraries depending on fklearn can still restrict the versions of the aforementioned libraries, keeping the previous behavior.

Remaining problems or questions

Since many libraries were bumped, and the test coverage may not be sufficient, it is possible that things break. In that case, we will first try to fix the code, and as second attempt we will restrict the library version.

@codecov-commenter
Copy link

codecov-commenter commented Dec 28, 2021

Codecov Report

Merging #181 (05037a9) into master (3cd7bec) will decrease coverage by 0.44%.
The diff coverage is 92.84%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #181      +/-   ##
==========================================
- Coverage   94.69%   94.24%   -0.45%     
==========================================
  Files          25       32       +7     
  Lines        1507     1928     +421     
  Branches      203      258      +55     
==========================================
+ Hits         1427     1817     +390     
- Misses         48       76      +28     
- Partials       32       35       +3     
Impacted Files Coverage Δ
src/fklearn/causal/validation/cate.py 0.00% <0.00%> (ø)
src/fklearn/data/datasets.py 100.00% <ø> (ø)
src/fklearn/tuning/parameter_tuners.py 79.48% <ø> (ø)
src/fklearn/tuning/selectors.py 90.47% <ø> (ø)
src/fklearn/validation/validator.py 91.17% <71.42%> (-3.11%) ⬇️
src/fklearn/preprocessing/splitting.py 95.00% <92.59%> (-0.84%) ⬇️
src/fklearn/training/calibration.py 96.36% <94.73%> (-3.64%) ⬇️
src/fklearn/training/transformation.py 93.95% <95.12%> (+0.02%) ⬆️
src/fklearn/training/regression.py 97.91% <95.45%> (-0.45%) ⬇️
src/fklearn/validation/evaluators.py 93.95% <96.29%> (+4.32%) ⬆️
... and 15 more

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 6aed21c...05037a9. Read the comment docs.

@gcbeltramini gcbeltramini marked this pull request as ready for review December 30, 2021 00:56
@gcbeltramini gcbeltramini requested a review from a team as a code owner December 30, 2021 00:56
@gcbeltramini gcbeltramini merged commit aa558fb into master Dec 30, 2021
@gcbeltramini gcbeltramini deleted the gcbeltramini-patch-1 branch December 30, 2021 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants