parknate@

Results 10 comments of parknate@

@trungleduc will you resolve the merge conflicts?

@akuma12 On-call here taking a look at this PR. I started approval workflow for tests. Are all code changes done here?

Hello, do you require any support on this PR?

@jmahlik I think I can pick it up, but can you provide a high-level overview of what you mean by "load the json and pick the max version"? To my...

@bzheng06 can you force push a temp commit to restart the workflow?

@akrishna1995 Can you take a look at this PR again? Some conflicts

Are you still facing the same issue with version 2.251.0?

./src/sagemaker/serve/builder/model_builder.py:115:1: F401 'sagemaker.utils.unique_name_from_base' imported but unused