awslabs / benchmark-ai

Anubis (formerly known as Benchmark AI), measures the goodness of machine learning workloads
Apache License 2.0
16 stars 6 forks source link

[SM-Executor] Adds support for 'args' for SageMaker benchmarks #1012

Closed perdasilva closed 3 years ago

perdasilva commented 4 years ago

Issue #, if available:

Description of changes: The SageMaker executor would ignore the ml.args configuration in the toml. This PR updates the sm-executor to pass the arguments to the underlying command being executed.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

perdasilva commented 4 years ago

@surajkota @haohanchen-yagao could you please give your ok here as well?

perdasilva commented 4 years ago

Maybe @haohanchen-yagao, could you please check out this branch and try it out. See if it at least improves things a bit?

perdasilva commented 3 years ago

Closing due to lack of response and age