Skip to content

Add Docker environment & web demo #9

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

Merged
merged 2 commits into from
Nov 4, 2021
Merged

Add Docker environment & web demo #9

merged 2 commits into from
Nov 4, 2021

Conversation

ericguizzo
Copy link
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
Copy link
Owner

f90 commented Oct 5, 2021

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
Copy link
Owner

f90 commented Oct 22, 2021

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
Copy link
Owner

f90 commented Oct 22, 2021

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
Copy link
Owner

f90 commented Oct 28, 2021

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

@ericguizzo
Copy link
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