Zeke Sikelianos comments

Results 472 comments of


                                            Zeke Sikelianos

Replace a naughty word with "something"

Yeah the app is just using the prediction object for state, so there's no storing of one string and predicting with another.

Add comments to the code

Thanks for opening this PR, but I'm not sure how useful it is. See https://github.com/replicate/scribble-diffusion/pull/22#issuecomment-1446980525

This is working locally but the preview app seems to be failing. ``` index-153743c76e8d4f30.js:1 POST https://scribble-diffusion-git-use-replicate-npm-package-replicate.vercel.app/api/predictions 500 ```

use the replicate npm package

``` $ vercel logs -f "https://scribble-diffusion-git-use-replicate-npm-package-replicate.vercel.app/" ``` Output: ``` 2023-02-25T06:36:38.252Z 2023-02-25T06:36:38.837Z c48daf0d-9fff-46cc-ac61-e7fb9411a31e ERROR Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'node-fetch' imported from /var/task/node_modules/replicate/lib/ReplicateClient.js 2023-02-25T06:36:38.252Z at new NodeError (node:internal/errors:393:5) 2023-02-25T06:36:38.252Z at packageResolve...

use the replicate npm package

I wonder if https://github.com/vercel/fetch is injecting itself into this somehow and screwing things up...

use the replicate npm package

@erbridge any idea what might be wrong here?

use the replicate npm package

@erbridge no it's still broken when you try to create a scribble.

use the replicate npm package

It looks like we're using other things from `node-fetch` besides just `fetch` like `FetchError`, `Response`, etc. @gr2m can you point to any good reference implementations that are using built-in fetch?...

use the replicate npm package

Works on my machine. Now with webhooks. Thanks @erbridge for unblocking all the things. ``` 🪝 received webhook for prediction: prv72qvwdfcdxalsypyjjlruye ``` ---

use the replicate npm package

This is ready for a review.