Merlin icon indicating copy to clipboard operation
Merlin copied to clipboard

Update merlin-tensorflow image to 23.08 in sagemaker example

Open edknv opened this issue 2 years ago • 4 comments

This PR updates the sagemaker example to use the latest merlin-tensorflow:23.08 image. Due to changes in triton/systems,

serve file that ships with the triton container is patched and we build our container with this patched version. Specifically, lines 143-167 that were used for disallowing multiple models are removed, and also --model-control-mode=explicit is removed from the last line.

Note also that the training script train.py is updated to reflect the latest changes in the nvtabular workflow.

edknv avatar Jul 03 '23 04:07 edknv

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

copy-pr-bot[bot] avatar Oct 14 '23 07:10 copy-pr-bot[bot]

@jperez999 would you mind reviewing the changes? thanks!

rnyak avatar Oct 26 '23 13:10 rnyak