try fix CI builds (#53)

* try fix builds * partial rollback * fix build dependency * make formatting mandatory & reformat kaplan meier test * add unreleased note * edited * adding sphinx-issues for Changelog links * reverted change --------- Co-authored-by: corolth1 <[email protected]>
Novartis · Sep 26, 2024 · 4e4d8fb · 4e4d8fb
1 parent 723192f
commit 4e4d8fb
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 7 deletions.
diff --git a/.github/workflows/build-release.yml b/.github/workflows/build-release.yml
@@ -27,7 +27,7 @@ jobs:
         run: python -m twine check dist/*
 
       - name: Upload artifact
-        uses: actions/upload-artifact@v2
+        uses: actions/upload-artifact@v4
         with:
           name: Python-package
           path: dist

diff --git a/.github/workflows/codeqc.yml b/.github/workflows/codeqc.yml
@@ -46,8 +46,8 @@ jobs:
       shell: bash -l {0}
       run: |
         conda activate torchsurv
-        ./dev/codeqc.sh
- 
+        ./dev/codeqc.sh check
+
     - name: Tests
       shell: bash -l {0}
       run: |

diff --git a/dev/environment.yml b/dev/environment.yml
@@ -3,7 +3,7 @@ channels:
   - conda-forge
   - pytorch
 dependencies:
-  - build=0.7.0
+  - python-build=1.2.2
   - pep517=0.13.0
   - numpy=1.26.4
   - pandas=2.2.0

diff --git a/docs/CHANGELOG.md b/docs/CHANGELOG.md
@@ -1,6 +1,16 @@
 Change log
 =========
 
+Version 0.1.3 (unreleased)
+--------------------------
+
+* Tutorial dataset error on momentum.ipynb #50
+* Fix issue #48 - log_hazard returns torch.Inf
+* Fix warning with Spearman correlation #41
+* Added in-depth statistical background to link AUC to C-index #39
+* Created Conda Forge version #47
+* Updated CICD builds #53
+
 Version 0.1.2
 -------------
 

diff --git a/tests/test_kaplan_meier.py b/tests/test_kaplan_meier.py
@@ -208,9 +208,9 @@ def test_kaplan_meier_prediction_error_raised(self):
         for batch in batch_container.batches:
             (train_time, train_event, test_time, *_) = batch
 
-            train_event[
-                -1
-            ] = False  # if last event is censoring, the last KM is > 0 and it cannot predict beyond this time
+            train_event[-1] = (
+                False  # if last event is censoring, the last KM is > 0 and it cannot predict beyond this time
+            )
             km = KaplanMeierEstimator()
             km(train_event, train_time, censoring_dist=False)