Skip to content

S3 Algos #46

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

Closed
wants to merge 7 commits into from
Closed

S3 Algos #46

wants to merge 7 commits into from

Conversation

ragavvenkatesan
Copy link

No description provided.

@ragavvenkatesan
Copy link
Author

ragavvenkatesan commented Jan 17, 2018

@orchidmajumder , let's use this PR to track the integration. Once done, we can consider merging to master.

num_layers = hp('num_layers', (isin(18, 34, 50, 101, 152, 200, 20, 32, 44, 56, 110), isint), \
'num_layers should be in the set [18, 34, 50, 101, 152, 200, 20, 32, 44, 56, 110]' )
resize = hp('resize', (gt(1), isint), 'resize should be an integer greater-than 1')
epochs = hp('epochs', (ge(1), isint), 'epochs should be an integer greater-than 1')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please note that isint() was removed, and the hp()'s have changed a bit. For reference see:
#54

@ragavvenkatesan
Copy link
Author

Closing this to follow the developer protocol for PRs.

laurenyu added a commit to laurenyu/sagemaker-python-sdk that referenced this pull request May 31, 2018
apacker pushed a commit to apacker/sagemaker-python-sdk that referenced this pull request Nov 15, 2018
Add license statement to input.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants