Skip to content

Commit

Permalink
Fix postcommit python arm workflow (#33799)
Browse files Browse the repository at this point in the history
* Fix postcommit python arm workflow

* formatting

* formatting
  • Loading branch information
akashorabek authored Jan 29, 2025
1 parent 13310b3 commit 3c563e8
Showing 1 changed file with 7 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,9 @@ def file_lines_sorted(filepath):
@pytest.mark.uses_sklearn
@pytest.mark.it_postcommit
class SklearnInference(unittest.TestCase):

# TODO(https://github.com/apache/beam/issues/33796) use older numpy
@unittest.skipIf((3, 9, 0) <= sys.version_info < (3, 11, 0), "Beam#33796")
def test_sklearn_mnist_classification(self):
test_pipeline = TestPipeline(is_integration_test=True)
input_file = 'gs://apache-beam-ml/testing/inputs/it_mnist_data.csv'
Expand Down Expand Up @@ -85,6 +88,8 @@ def test_sklearn_mnist_classification(self):
true_label, expected_prediction = expected_outputs[i].split(',')
self.assertEqual(predictions_dict[true_label], expected_prediction)

# TODO(https://github.com/apache/beam/issues/33796) use older numpy
@unittest.skipIf((3, 9, 0) <= sys.version_info < (3, 11, 0), "Beam#33796")
def test_sklearn_mnist_classification_large_model(self):
test_pipeline = TestPipeline(is_integration_test=True)
input_file = 'gs://apache-beam-ml/testing/inputs/it_mnist_data.csv'
Expand Down Expand Up @@ -118,7 +123,8 @@ def test_sklearn_mnist_classification_large_model(self):
self.assertEqual(predictions_dict[true_label], expected_prediction)

# TODO(https://github.com/apache/beam/issues/27151) use model with sklearn 1.2
@unittest.skipIf(sys.version_info >= (3, 11, 0), "Beam#27151")
# TODO(https://github.com/apache/beam/issues/33796) use older numpy
@unittest.skipIf(sys.version_info >= (3, 9, 0), "Beam#27151")
def test_sklearn_regression(self):
test_pipeline = TestPipeline(is_integration_test=True)
input_file = 'gs://apache-beam-ml/testing/inputs/japanese_housing_test_data.csv' # pylint: disable=line-too-long
Expand Down

0 comments on commit 3c563e8

Please sign in to comment.