Skip to content

feat: Support placeholders for TransformStep #157

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.

Sign up for GitHub

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

Merged
merged 13 commits into from
Oct 18, 2021
Merged

feat: Support placeholders for TransformStep #157

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
b197594
Support placeholders for transfor step
ca-nguyen Sep 1, 2021
9ff7624
Use max_payload in execution input
ca-nguyen Sep 2, 2021
8892552
Merge branch 'main' into support-placeholders-for-transform-step
ca-nguyen Sep 2, 2021
4ad008f
Adjust number of concurrent transform
ca-nguyen Sep 3, 2021
bbe0f9e
Added parameters arg doc
ca-nguyen Sep 3, 2021
e5d1eca
Merge branch 'main' into support-placeholders-for-transform-step
shivlaks Sep 10, 2021
44ff061
Merge branch 'main' into support-placeholders-for-transform-step
shivlaks Sep 11, 2021
52d8854
Merge branch 'main' into support-placeholders-for-transform-step
ca-nguyen Sep 13, 2021
58b1685
Add retry to integ test
ca-nguyen Sep 13, 2021
d575518
Updated tests
ca-nguyen Sep 15, 2021
f16041b
Merge branch 'main' into support-placeholders-for-transform-step
ca-nguyen Oct 15, 2021
3d7970a
Apply suggestions from code review
ca-nguyen Oct 15, 2021
1422243
Correct indentation and use placeholder for experiment_config in unit…
ca-nguyen Oct 15, 2021
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Adjust number of concurrent transform
  • Loading branch information
@ca-nguyen
ca-nguyen committed Sep 3, 2021
commit 4ad008f8ee38616e8b020ab2f97ce34e36cc055d
6 changes: 3 additions & 3 deletions tests/integ/test_sagemaker_steps.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -243,11 +243,11 @@ def test_transform_step_with_placeholder(trained_estimator, sfn_client, sfn_role
'model_name': job_name,
'data': transform_input,
'content_type': "text/csv",
'instance_count': 1,
'instance_type': "ml.m5.large",
'instance_count': INSTANCE_COUNT,
'instance_type': INSTANCE_TYPE,
'split_type': 'Line',
'strategy': 'SingleRecord',
'max_concurrent_transforms': 2,
'max_concurrent_transforms': 1,
'max_payload': 5
}

Expand Down