-
Notifications
You must be signed in to change notification settings - Fork 111
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
[BUG] Update Training and Serving Merlin on AWS SageMaker using latest merlin image #1039
Comments
I'm working on updating the After bumping the image version to
which doesn't tell us much what is going wrong. I'll try to run the container locally to debug. |
thanks @edknv ! |
@edknv are there any update on this issue? I am trying to deploy a Merlin model to Sagemaker following the example given. I am running into the same issue. |
@wei-m-teh Apologies for the delay. It's in review at the moment, but I updated #1040 with a workaround I found for making the notebook work with the latest |
@wei-m-teh can you please test this PR at your end? thanks. |
Bug description
One of our customers are having an issue of reproducing the training and serving Merlin on AWS SM example and they get an error (will be provided eventually).
The documentation also should be improved/clarified since it is not clear how one can generate the dataset in
Generating Dataset
without installing Merlin libs, and using Merlin image.Steps/Code to reproduce bug
The notebooks should be tested with the latest stable merlin-tensorflow image, and updated if required. Currently, in the example
merlin-tensorflow:22.10
image is used.Expected behavior
Environment details
Additional context
The text was updated successfully, but these errors were encountered: