Skip to content

Commit 5211801

Browse files
authored
Fix Ray Train release test (#51624)
## Why are these changes needed? <!-- Please give a short summary of the change and the problem this solves. --> * Fix Ray Train release test Arg parsing. * Bump up limit from 500K to 1M rows. Signed-off-by: Srinath Krishnamachari <srinath.krishnamachari@anyscale.com>
1 parent 35d472a commit 5211801

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

release/train_tests/benchmark/config.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ class BenchmarkConfig(BaseModel):
5050
num_epochs: int = 1
5151
skip_train_step: bool = False
5252
train_step_anomaly_detection: bool = False
53-
limit_training_rows: int = 500000
53+
limit_training_rows: int = 1000000
5454

5555
# Validation
5656
validate_every_n_steps: int = -1
@@ -72,8 +72,8 @@ def _add_field_to_parser(parser: argparse.ArgumentParser, field: str, field_info
7272
if field_type is bool:
7373
parser.add_argument(
7474
f"--{field}",
75-
type=lambda x: x.lower() == "true",
76-
default=field_info.default,
75+
action="store_true",
76+
help=f"Enable {field} (default: {field_info.default})",
7777
)
7878
else:
7979
parser.add_argument(f"--{field}", type=field_type, default=field_info.default)

0 commit comments

Comments
 (0)