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

Port project to Github Actions and bump dependencies to better support python 3.9 #225

Closed
wants to merge 35 commits into from

Conversation

fpingas
Copy link
Contributor

@fpingas fpingas commented Mar 20, 2023

Status

READY

Todo list

  • Documentation
  • Tests added and passed

Background context

CircleCI is not working consistently and the package takes ages to install in python 3.9 due to its dependency locks.

Description of the changes proposed in the pull request

Port CircleCI workflows to Github Action workflows, one push, for running test and code checks at every push for code quality, and one publish, for publishing the package when a new github release is published.
This PR also updates the maximum allowed scikit-learn dependency so it installs quicker in 3.9.

@codecov-commenter
Copy link

codecov-commenter commented Mar 20, 2023

Codecov Report

Merging #225 (2a8a099) into master (3cd7bec) will decrease coverage by 0.48%.
The diff coverage is 93.05%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##           master     #225      +/-   ##
==========================================
- Coverage   94.69%   94.21%   -0.48%     
==========================================
  Files          25       34       +9     
  Lines        1507     2109     +602     
  Branches      203      281      +78     
==========================================
+ Hits         1427     1987     +560     
- Misses         48       84      +36     
- Partials       32       38       +6     
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/training/unsupervised.py 92.30% <60.00%> (-7.70%) ⬇️
src/fklearn/validation/validator.py 91.20% <88.88%> (-3.08%) ⬇️
src/fklearn/preprocessing/splitting.py 95.00% <92.59%> (-0.84%) ⬇️
src/fklearn/causal/cate_learning/meta_learners.py 93.02% <93.02%> (ø)
src/fklearn/training/calibration.py 96.36% <94.73%> (-3.64%) ⬇️
src/fklearn/training/transformation.py 93.95% <95.12%> (+0.02%) ⬆️
... and 15 more

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

setup.py Show resolved Hide resolved
Comment on lines +4 to +6
on:
release:
types: [published]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a change in behavior, but seems OK: publish the package in PyPI when there is a published release on GitHub. We can test how this will work with tags and protected branches.

@fpingas fpingas changed the title demo workflow Port project to Github Actions and bump dependencies to better support python 3.9 Mar 27, 2023
@fpingas
Copy link
Contributor Author

fpingas commented Mar 28, 2023

Implemented in #227

@fpingas fpingas closed this Mar 28, 2023
@fpingas fpingas deleted the fp/workflow branch March 28, 2023 15:13
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