From ff7c070f5086c1b32bb8de9a81c6ed4853c8d967 Mon Sep 17 00:00:00 2001 From: Raphael Meudec Date: Mon, 21 Mar 2022 11:43:55 +0100 Subject: [PATCH 1/4] Drop before 2.3.0, extend to 2.8.0 --- .github/workflows/ci.yml | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 8ec6a0d..e9b6c76 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -41,10 +41,7 @@ jobs: strategy: matrix: python-version: [3.6, 3.7, 3.8] - tensorflow-version: [2.1.0, 2.2.0, 2.3.0, 2.4.0, 2.5.0] - exclude: - - tensorflow-version: 2.1.0 - python-version: 3.8 + tensorflow-version: [2.3.0, 2.4.0, 2.5.0] steps: - uses: actions/checkout@v2 @@ -70,9 +67,15 @@ jobs: runs-on: ubuntu-latest strategy: matrix: - python-version: [3.6, 3.7, 3.8] - tensorflow-version: [2.6.0] - keras-version: [2.6.0] + python-version: [3.7, 3.8] + tensorflow-version: [2.6.0, 2.7.0, 2.8.0] + include: + - tensorflow-version: 2.6.0 + keras-version: 2.6.0 + - tensorflow-version: 2.7.0 + keras-version: 2.7.0 + - tensorflow-version: 2.8.0 + keras-version: 2.8.0 steps: - uses: actions/checkout@v2 From 4a809d6e3c7cda4a2d7b0c1f92e6900312e975ea Mon Sep 17 00:00:00 2001 From: Raphael Meudec Date: Mon, 21 Mar 2022 12:02:17 +0100 Subject: [PATCH 2/4] Smaller dataset in CI --- tests/integration/conftest.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/integration/conftest.py b/tests/integration/conftest.py index 340d40a..0dfafaf 100644 --- a/tests/integration/conftest.py +++ b/tests/integration/conftest.py @@ -23,10 +23,10 @@ def mnist_dataset(num_classes): test_labels = tf.keras.utils.to_categorical(test_labels, num_classes=num_classes) return ( - train_images[0:500], - train_labels[0:500], - test_images[0:300], - test_labels[0:300], + train_images[0:100], + train_labels[0:100], + test_images[0:50], + test_labels[0:50], ) From f6e1c72b0a85e4ee33bc95f7aa50c6d11115a088 Mon Sep 17 00:00:00 2001 From: Raphael Meudec Date: Mon, 21 Mar 2022 13:08:10 +0100 Subject: [PATCH 3/4] Try without parallelization? Tests work fine locally --- .github/workflows/ci.yml | 2 +- tests/integration/test_keras_api.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index e9b6c76..7797088 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -66,6 +66,7 @@ jobs: runs-on: ubuntu-latest strategy: + max-parallel: 1 matrix: python-version: [3.7, 3.8] tensorflow-version: [2.6.0, 2.7.0, 2.8.0] @@ -76,7 +77,6 @@ jobs: keras-version: 2.7.0 - tensorflow-version: 2.8.0 keras-version: 2.8.0 - steps: - uses: actions/checkout@v2 - name: Set up Python ${{ matrix.python-version }} diff --git a/tests/integration/test_keras_api.py b/tests/integration/test_keras_api.py index 71114cd..d637ef0 100644 --- a/tests/integration/test_keras_api.py +++ b/tests/integration/test_keras_api.py @@ -173,4 +173,4 @@ def test_all_keras_api( ] # Start training - model.fit(train_images, train_labels, epochs=3, callbacks=callbacks) + model.fit(train_images, train_labels, epochs=2, callbacks=callbacks) From 1a83e4f9b7d973a2ed294444d4bf974240b8973e Mon Sep 17 00:00:00 2001 From: Raphael Meudec Date: Mon, 21 Mar 2022 13:24:13 +0100 Subject: [PATCH 4/4] Maybe something weird happening with parallelization that mix up the graph? --- .github/workflows/ci.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 7797088..043b9c0 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -39,6 +39,7 @@ jobs: runs-on: ubuntu-latest strategy: + max-parallel: 1 matrix: python-version: [3.6, 3.7, 3.8] tensorflow-version: [2.3.0, 2.4.0, 2.5.0]