-
Notifications
You must be signed in to change notification settings - Fork 27.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CI: AMD MI300 tests fix #30797
CI: AMD MI300 tests fix #30797
Changes from all commits
dfc174b
9a8032d
5650534
7da11b8
7841c76
b64b9d5
c55576b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2507,7 +2507,7 @@ def test_trainer_eval_mrpc(self): | |
) | ||
eval_dataset = GlueDataset(data_args, tokenizer=tokenizer, mode="dev") | ||
|
||
training_args = TrainingArguments(output_dir="./examples", use_cpu=True) | ||
training_args = TrainingArguments(output_dir="./examples", use_cpu=True, report_to="none") | ||
trainer = Trainer(model=model, args=training_args, eval_dataset=eval_dataset) | ||
result = trainer.evaluate() | ||
self.assertLess(result["eval_loss"], 0.2) | ||
|
@@ -2528,6 +2528,7 @@ def test_trainer_eval_multiple(self): | |
output_dir="./examples", | ||
use_cpu=True, | ||
per_device_eval_batch_size=1, | ||
report_to="none", | ||
) | ||
trainer = Trainer( | ||
model=model, | ||
|
@@ -3063,6 +3064,8 @@ def test_end_to_end_example(self): | |
"--predict_with_generate", | ||
"--ddp_timeout", | ||
"60", | ||
"--report_to", | ||
"none", | ||
Comment on lines
+3067
to
+3068
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. How come we have to add this? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Since |
||
] | ||
execute_subprocess_async(command) | ||
# successful return here == success - any errors would have caused an error or a timeout in the sub-call | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -161,7 +161,6 @@ def test_return_sequences(self): | |
tokenizer=tokenizer, | ||
data_collator=data_collator, | ||
compute_metrics=lambda x: {"samples": x[0].shape[0]}, | ||
report_to="none", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is this intended? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, the argument does not exist and error introduced from here: #30266 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok thanks ! |
||
) | ||
|
||
def prepare_data(examples): | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!