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

IBM→ BiomedSciAI Github org transfer adjustments #51

Merged
merged 23 commits into from
Dec 21, 2022

Conversation

ehudkr
Copy link
Collaborator

@ehudkr ehudkr commented Dec 21, 2022

Necessary changes after transferring causallib from the IBM org on github to the BiomedSciAI org.
Including:

  1. Migrating Travis CI to Github Actions
  2. Updating 3rd party services (e.g., Code Climate, readthedocs, PyPI...)
  3. Updating internal docs/examples referring to the Github repo

ehudkr and others added 23 commits December 19, 2022 09:22
Signed-off-by: Ehud-Karavani <ehud.karavani@ibm.com>
Don't cancel other the rest of the python version builds if only one (or more, just make them independent) fail.

It is not uncommon that only certain python versions fail due to dependency hell.
Since `hemm` failed due to no Pytorch
Test whether only one version succeeding is a matter of some resource parallel allocation mutex.
Revert the `fail-fast`, but now try to ensure that there are no parallel jobs by using `max-parallel`
Currently causes two main issues:
 - No `normalize` in `LinearRegression`.
   Easy, just need to update tests to use a Pipeline
 - Force singular feature type in pandas columns (harder)
    Harder, will require a safe-join mechanism when adding treatment as variable.
Workflow's name appear in the badge, so change that to a short `Build`
instead of a long "Build and install causallib".
Consequentially change the yml file from `test` to `build` to preserve
consistency

Signed-off-by: Ehud-Karavani <ehud.karavani@ibm.com>
Currently causes a problem with matplotlib when plotting `fill_between` in the calibration plot tests
Github Actions do not support python 3.6 on `ubuntu-latest`*, 
and since python 3.6 is well after its end of life, I see no use in hacking a way for it work.

Farewell python 3.6, you were great


*```
Error: The version '3.6' with architecture 'x64' was not found for Ubuntu 22.04.
The list of all available versions can be found here: https://raw.githubusercontent.com/actions/python-versions/main/versions-manifest.json
```
Signed-off-by: Ehud-Karavani <ehud.karavani@ibm.com>
Goodbye, Travis, thank you for your service.

Signed-off-by: Ehud-Karavani <ehud.karavani@ibm.com>
Check whether build now passes when removing the single process constraint and allowed to run in parallel
Signed-off-by: Ehud-Karavani <ehud.karavani@ibm.com>
@ehudkr ehudkr merged commit cbec306 into master Dec 21, 2022
@ehudkr ehudkr deleted the ibm2biomedsciai-transfer branch December 21, 2022 14:51
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.

2 participants