Skip to content

Fixed: payload was larger than SageMaker limit #224

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

Jump to bottom
Merged
merged 1 commit into from
Mar 31, 2018
Merged

Fixed: payload was larger than SageMaker limit #224

merged 1 commit into from
Mar 31, 2018

Conversation

djarpin
Copy link
Contributor

@djarpin djarpin commented Mar 31, 2018

No description provided.

@djarpin djarpin requested a review from mrnikwaws March 31, 2018 22:26
mrnikwaws
mrnikwaws approved these changes Mar 31, 2018
View reviewed changes
Copy link

@mrnikwaws mrnikwaws left a comment

Choose a reason for hiding this comment

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

I know 60 worked so 100 should be fine

@djarpin
Copy link
Contributor Author

djarpin commented Mar 31, 2018

Yes. I tested 100 and it worked. I figured going to an even slightly smaller payload (100 splits rather than 60 would give us some headroom/future-proofing).

@djarpin djarpin merged commit fb52f25 into master Mar 31, 2018
@djarpin djarpin deleted the arpin_pca_mnist_payload_size branch April 10, 2018 15:03
atqy pushed a commit to atqy/amazon-sagemaker-examples that referenced this pull request Aug 16, 2022
@rahul003
Delete xgboost.ipynb so that it can be renamed to xgboost_regression.…
ae6dd0b 
…ipynb (aws#224)
atqy added a commit to atqy/amazon-sagemaker-examples that referenced this pull request Nov 30, 2022
@atqy
Keynote2 rtd (aws#224)
ad2eb4a 
* remove --no-deps tags from notebook

* configure rst files for reinvent notebooks + fix any rtd leveling issues

* make corrections

* reformat
atqy added a commit to atqy/amazon-sagemaker-examples that referenced this pull request Nov 30, 2022
@atqy
Keynote2 rtd (aws#224)
e1e062c 
* remove --no-deps tags from notebook

* configure rst files for reinvent notebooks + fix any rtd leveling issues

* make corrections

* reformat
atqy added a commit that referenced this pull request Nov 30, 2022
@atqy @aberlm
@rodzanto @seanpmorgan @Zhenshan-Jin @billdoors @shreyapandit
Reinvent22 (#3686)
cf7012a 
* Adding example for native AutoML step in SageMaker Pipelines. (#211)

* Adding example for native AutoML step in SageMaker Pipelines.

* Adding SageMaker Geospatial example for Digital Farming (#213)

* Adding digital-farming-pipelines notebooks

* Adding digital-farming-pipelines notebooks

* Update pipelines-sagemaker-geospatial.ipynb

* Update pipelines-sagemaker-geospatial.ipynb

* Update pipelines-sagemaker-geospatial.ipynb

* Update pipelines-sagemaker-geospatial.ipynb

* Adding digital-farming-pipelines notebooks

* Adding digital-farming-pipelines notebooks

* Adding code folder

* Adding Lambda function

* Adjusting notebook

* Adjuting roles

* Adjusted roles

* Updated notebooks

* Updated notebooks format

* Adding Scheduled Notebook (Keynote2) (#216)

* * Add scheduled notebook example

* Update notebook

* Format

* Change image dir

* Resolve comments.

* Resolve comments.

* Resolve comments.

* Add example notebook for model governance model card (#217)

Co-authored-by: Zhenshan Jin <zsjin@amazon.com>

* shadow endpoint (#219)

* initial commit

* minor fix

* minor fixes

* add inference experiment notebook

add inference experiment notebook

* Add shadow endpoint notebook (#215)

* initial commit

* add inference experiment notebook

add inference experiment notebook

Co-authored-by: Qingwei Li<ec2-user@ip-172-16-37-37.us-west-2.compute.internal>
Co-authored-by: Shreya Pandit <pandishr@amazon.com>
Co-authored-by: Qingwei Li <qqnl@amazon.com>

* Revert "Add shadow endpoint notebook (#215)" (#218)

This reverts commit b6d2fd203f7f85670478556e902ad2bb86a1a882.

* reformat

* reviewer's comments addressed

* clear output

* fix and reformat nb

* reformat nb

* remove notebook

* markdown change

Co-authored-by: EC2 Default User <ec2-user@ip-172-16-37-37.us-west-2.compute.internal>
Co-authored-by: Shreya Pandit <pandishr@amazon.com>
Co-authored-by: Qingwei Li <qqnl@amazon.com>
Co-authored-by: EC2 Default User <ec2-user@ip-172-16-0-250.us-west-2.compute.internal>

* rename sagemaker-shadow-endpoint folder

* upgrade pip packages for reinvent

* install from reinvent wheels temporarily

* Keynote2 rtd (#224)

* remove --no-deps tags from notebook

* configure rst files for reinvent notebooks + fix any rtd leveling issues

* make corrections

* reformat

* updated shadow_variants with Alwin's work (#225)

* initial commit

* minor fix

* minor fixes

* add inference experiment notebook

add inference experiment notebook

* Add shadow endpoint notebook (#215)

* initial commit

* add inference experiment notebook

add inference experiment notebook

Co-authored-by: Qingwei Li<ec2-user@ip-172-16-37-37.us-west-2.compute.internal>
Co-authored-by: Shreya Pandit <pandishr@amazon.com>
Co-authored-by: Qingwei Li <qqnl@amazon.com>

* Revert "Add shadow endpoint notebook (#215)" (#218)

This reverts commit b6d2fd203f7f85670478556e902ad2bb86a1a882.

* reformat

* reviewer's comments addressed

* clear output

* fix and reformat nb

* reformat nb

* remove notebook

* markdown change

* Alwin's edit

add edits from Alwin

* reformat

* change folder name

Co-authored-by: EC2 Default User <ec2-user@ip-172-16-37-37.us-west-2.compute.internal>
Co-authored-by: Shreya Pandit <pandishr@amazon.com>
Co-authored-by: Qingwei Li <qqnl@amazon.com>
Co-authored-by: EC2 Default User <ec2-user@ip-172-16-0-250.us-west-2.compute.internal>
Co-authored-by: atqy <atqy@amazon.com>
Co-authored-by: atqy <95724753+atqy@users.noreply.github.com>

* add region tag to allow download from opt-in regions (#226)

* add region tag to allow download from opt-in regions

* add region tag to allow download from opt-in regions

* Trigger Build

Co-authored-by: Marcelo Aberle <116175523+aberlm@users.noreply.github.com>
Co-authored-by: Antonio Rodriguez <42835728+rodzanto@users.noreply.github.com>
Co-authored-by: Sean Morgan <seanmorgan91@gmail.com>
Co-authored-by: Zhenshan Jin <zhenshan.jin.jz@gmail.com>
Co-authored-by: Zhenshan Jin <zsjin@amazon.com>
Co-authored-by: Qingwei Li <billdoors@users.noreply.github.com>
Co-authored-by: EC2 Default User <ec2-user@ip-172-16-37-37.us-west-2.compute.internal>
Co-authored-by: Shreya Pandit <pandishr@amazon.com>
Co-authored-by: Qingwei Li <qqnl@amazon.com>
Co-authored-by: EC2 Default User <ec2-user@ip-172-16-0-250.us-west-2.compute.internal>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mrnikwaws mrnikwaws mrnikwaws approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@djarpin @mrnikwaws