Support ListTrials by trial component name #67
Conversation
| api_types.TrialSummary.from_boto, | ||
| "TrialSummaries", | ||
| experiment_name=experiment_name, | ||
| trial_component_name=trial_component_name, |
There was a problem hiding this comment.
does this depend on a new boto version? please verify min boto version specified in dependencies has this change
There was a problem hiding this comment.
from https://github.com/boto/boto3/blob/2a4efd21345597189fbe3cccaa4b5593da504cb1/.changes/1.10.20.json looks like last update on sagemaker client is happening in 1.10.20 version
There was a problem hiding this comment.
actually mini boto3 version is https://boto3.amazonaws.com/v1/documentation/api/1.12.8/reference/services/sagemaker.html#SageMaker.Client.list_trials
There was a problem hiding this comment.
looks like min boto version needs to be bumped then
https://github.com/aws/sagemaker-experiments/blob/master/setup.py#L27
2 participants
Issue #, if available:
Description of changes:
Testing done:
Merge Checklist
Put an
xin the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.General
Tests
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.