Ewe Zi Yi comments

Results 7 comments of


                                            Ewe Zi Yi

[BUG] Fast registration removes datetime metadata from files

Thanks @ChungYujoyce for the PR and @eapolinario for taking a look at this. The reason why we brought this up was because one of our users was attempting to zip...

chore(api): Update go dependencies in merlin api server

Thanks for the reminder; I just rebased my branch! :D So many PRs being merged during this short time x_x

chore(api): Update go dependencies in merlin api server

Thanks a lot for the review @ariefrahmansyah ! :D I'll be merging this now!

feat(mlflow): Add s3 artifact type

It seems like the codecov report is getting a lot of unexpected coverage drops and I think it might be due to the way we are using codecov: https://docs.codecov.com/docs/unexpected-coverage-changes. I...

feat(mlflow): Add s3 artifact type

Thanks for the review @tiopramayudi ! I'll be merging this PR shortly!

feat(api): Refactor API server to introduce support for Docker image registries and s3 bucket support

Thanks a lot for the comments @tiopramayudi and @bthari ! I should've covered and addressed all of the outstanding comments. If all looks good feel free to approve it again...

feat(api): Refactor API server to introduce support for Docker image registries and s3 bucket support

Thanks for the re-review @bthari! I'll keep this PR open a little while longer if Tio wants to look at the part I added to the Merlin SDK to allow...