👷 ci: remove LFS

.gitattributes

@@ -1 +0,0 @@
-Tests/MAExamples/Base/data/**/* filter=lfs diff=lfs merge=lfs -text

.github/workflows/examples.yml

@@ -16,10 +16,6 @@ jobs:
 
     steps:
       - uses: actions/checkout@v3
-        with:
-          lfs: true
-      - name: Pull lfs
-        run: git lfs pull
 
       - name: Setup Conda Environment
         run: |

Docs/Contributing/CHANGELOG.md

@@ -4,6 +4,7 @@ All notable changes to this project will be documented in this file.
 
 ## [unreleased]
 
+👷 **ci:** remove LFS ([#17](https://github.com/owkin/MAKit/pull/17))\
 🔧 **chore:** rollback changelog ([#16](https://github.com/owkin/MAKit/pull/16))\
 🔧 **chore:** fix badges in the readme ([#15](https://github.com/owkin/MAKit/pull/15))
 

Docs/Contributing/CONTRIBUTING.md

@@ -246,12 +246,13 @@ containing the commits to merge into the `main` branch.
       Do not delete the "Unreleased" section title: future PRs will insert 
       changelog items in this section.
     - Commit and push the changes.
-    - Squash and merge the new branch into `release_N` with commit message \
-      🔧 chore: release X.Y.Z
+    - Squash and merge the new branch into `release_N`.
 
 1. Create a Pull Request
 
-1. Review and Merge the Pull Request
+1. Review and Merge the Pull Request, change the commit 
+   message \
+   🔧 chore: release X.Y.Z
 
 1. Create a GitHub release X.Y.Z from `main`: 
      - GitHub > Releases > Draft new Release

Docs/Examples/VGG.md

@@ -80,18 +80,6 @@ conda deactivate
 conda env remove --name maexamples
 ```
 
-## Known Issues
-
-The following error may follow on local setup: 
-`UnpicklingError: invalid load key, ‘v’`.
-
-This may occur when lfs data files are not pulled. 
-To fix this: 
-
-```bash     
-git lfs pull
-```
-
 ## Steps
 
 1. Dump the training and testing datasets.

Sources/MATestsUtils/Trainer.swift

@@ -6,7 +6,6 @@
 //
 
 import Foundation
-import XCTest
 import MAKit
 
 ///
@@ -68,13 +67,19 @@ extension TestError: CustomStringConvertible
 ///
 /// Function used to retry flaky numeric tests.
 ///
-/// This function ensures test fails when internal function did not complete.
+/// This function is used to assert failure when internal function did not complete.
+/// Note that we do not import XCTest to avoid linking difficulties when importing MAKit into
+/// other projects. Hence the use of blockFail.
 ///
 /// - Parameters:
 ///     - nbRetry: Number maximal of retries.
 ///     - block: Function to execute.
+///     - blockFail: Function to execute in case of fail.
 ///
-public func retryNumeric(nbRetry: Int, _ block: @escaping () throws -> ())
+public func retryNumeric(
+    nbRetry: Int,
+    _ block: @escaping () throws -> (),
+    _ blockFail: ()->())
 {
     var iter = 0
     while iter < nbRetry
@@ -93,7 +98,7 @@ public func retryNumeric(nbRetry: Int, _ block: @escaping () throws -> ())
     }
     if iter == nbRetry
     {
-        XCTAssert(false)
+        blockFail()
     }
 }