Skip to content

Add Docker environment & web demo#9

Merged
f90 merged 2 commits into
f90:masterfrom
ericguizzo:master
Nov 4, 2021
Merged

Add Docker environment & web demo#9
f90 merged 2 commits into
f90:masterfrom
ericguizzo:master

Conversation

@ericguizzo

Copy link
Copy Markdown
Contributor

Hey @f90!

I really appreciate your wave-u-net implementation!

This pull request makes it possible to run your model inside a Docker environment, which makes it easier for other people to run it. We're using an open source tool called Cog to make this process easier.

This also means we can make a web page where other people can try out your model! View it here: https://replicate.ai/f90/wave-u-net-pytorch

Claim your page here so you can edit it, and we'll feature it on our website and tweet about it too.

In case you're wondering who I am, I'm from Replicate, where we're trying to make machine learning reproducible. We got frustrated that we couldn't run all the really interesting ML work being done. So, we're going round implementing models we like. 😊

@ericguizzo ericguizzo force-pushed the master branch 4 times, most recently from a392ff1 to e25da3a Compare September 29, 2021 15:36
@f90

f90 commented Oct 5, 2021

Copy link
Copy Markdown
Owner

Hey, thanks for the pull request. Looks exciting! I will take a look at it on the weekend, is there anything on my end I need to do code-wise to merge this successfully or do I just need to review?

@f90

f90 commented Oct 22, 2021

Copy link
Copy Markdown
Owner

Looking good, I suppose if I change/improve the model in the master branch it will automatically be improving on your website as well, or how does that work? @ericguizzo

@f90

f90 commented Oct 22, 2021

Copy link
Copy Markdown
Owner

Btw I claimed the model on your website you were linking to, seems to work already :) Nice job, hope you'll be very successful 👍

@f90

f90 commented Oct 28, 2021

Copy link
Copy Markdown
Owner

Pinging @ericguizzo , I can just merge this now and everything works right?

@ericguizzo

Copy link
Copy Markdown
Contributor Author

Hello f90, sorry for the late response! Thank you very much for merging the pull request and for claiming the model!
Everything seems to work fine :)

@f90 f90 merged commit 443fe30 into f90:master Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants