Ahmed Kamel

Results 28 comments of Ahmed Kamel

Hi team, any updates on this?

@corymhall with the comments addressed, anything else u think should be tackled before moving forwards with this PR?

How different is this from renaming logical ids? Is the difference that relocating can happen at some intermediate construct level therefore might save multiple logical Id renames?

Hi there... any updates on this?

Thanks @ijemmy and @dreamorosi for looking into this. > What do you think? I do like the `Recommended practices` and updating the `examples`. As a Powertools user, this could've helped...

This is something my team would be interested in as well. The Neptune Bulk Loader is all about S3 https://docs.aws.amazon.com/neptune/latest/userguide/bulk-load.html There's basically 2 APIs 1. One to request the loading....

I've submitted 2 near-identical PRs to both `sagemaker-training-toolkit` and `sagemaker-inference-toolkit` at - https://github.com/aws/sagemaker-inference-toolkit/pull/130 - https://github.com/aws/sagemaker-training-toolkit/pull/187 My understanding is that if these get merged, then new containers leveraging those packages should...

I've also submitted a pr for `sagemaker-pytorch-inference-toolkit` at https://github.com/aws/sagemaker-pytorch-inference-toolkit/pull/150

Inference-side changes have been merged - https://github.com/aws/sagemaker-inference-toolkit/pull/130 - https://github.com/aws/sagemaker-pytorch-inference-toolkit/pull/150