SUMIN/HuggingFace_transformer
1
0
Fork 0
Code Issues Pull Requests Actions 7 Packages Projects Releases Wiki Activity

Skipping more high mem tests - Wav2Vec2 Hubert (#21647)

Browse Source 
Skipping more tests
This commit is contained in:
amyeroberts
2023-02-15 16:00:50 +00:00
committed by GitHub
parent 0c9c8472e6
commit 3499c49c17
2 changed files with 12 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
tests/models/hubert/test_modeling_tf_hubert.py
View File

@@ -321,19 +321,15 @@ class TFHubertModelTest(TFModelTesterMixin, unittest.TestCase):
model = TFHubertModel.from_pretrained("facebook/hubert-base-ls960") model = TFHubertModel.from_pretrained("facebook/hubert-base-ls960")
self.assertIsNotNone(model) self.assertIsNotNone(model)
# We override here as passing a full batch of 13 samples results in OOM errors for CTC @unittest.skip(reason="Fix me! Hubert hits OOM errors when loss is computed on full batch")
def test_dataset_conversion(self): def test_dataset_conversion(self):
default_batch_size = self.model_tester.batch_size # TODO: (Amy) - check whether skipping CTC model resolves this issue and possible resolutions for CTC
self.model_tester.batch_size = 2 pass
super().test_dataset_conversion()
self.model_tester.batch_size = default_batch_size
# We override here as passing a full batch of 13 samples results in OOM errors for CTC @unittest.skip(reason="Fix me! Hubert hits OOM errors when loss is computed on full batch")
def test_keras_fit(self): def test_keras_fit(self):
default_batch_size = self.model_tester.batch_size # TODO: (Amy) - check whether skipping CTC model resolves this issue and possible resolutions for CTC
self.model_tester.batch_size = 2 pass
super().test_keras_fit()
self.model_tester.batch_size = default_batch_size
@require_tf @require_tf

17
tests/models/wav2vec2/test_modeling_tf_wav2vec2.py
View File

@@ -512,20 +512,15 @@ class TFWav2Vec2RobustModelTest(TFModelTesterMixin, unittest.TestCase):
model = TFWav2Vec2Model.from_pretrained("facebook/wav2vec2-base-960h") model = TFWav2Vec2Model.from_pretrained("facebook/wav2vec2-base-960h")
self.assertIsNotNone(model) self.assertIsNotNone(model)
# We override here as passing a full batch of 13 samples results in OOM errors for CTC @unittest.skip(reason="Fix me! Wav2Vec2 hits OOM errors when loss is computed on full batch")
@unittest.skip("Fix me!")
def test_dataset_conversion(self): def test_dataset_conversion(self):
default_batch_size = self.model_tester.batch_size # TODO: (Amy) - check whether skipping CTC model resolves this issue and possible resolutions for CTC
self.model_tester.batch_size = 2 pass
super().test_dataset_conversion()
self.model_tester.batch_size = default_batch_size
# We override here as passing a full batch of 13 samples results in OOM errors for CTC @unittest.skip(reason="Fix me! Wav2Vec2 hits OOM errors when loss is computed on full batch")
def test_keras_fit(self): def test_keras_fit(self):
default_batch_size = self.model_tester.batch_size # TODO: (Amy) - check whether skipping CTC model resolves this issue and possible resolutions for CTC
self.model_tester.batch_size = 2 pass
super().test_keras_fit()
self.model_tester.batch_size = default_batch_size
@require_tf @require_tf