Zeke Sikelianos comments

Results 472 comments of


                                            Zeke Sikelianos

added cog.yaml warning

Thanks for the PR. It's a little confusing that this warning describes two possible issues. What do you think about handling the two exceptions with different warnings? Some untested pseudocode:...

added cog.yaml warning

So... this will print a warning, but it will also blow up because the `pass` was removed, right? Is that that the intended behavior here, or do we want to...

Multiple output prediction is possible in cog?

Yep, you wanna do something like this: ```python from cog import BasePredictor, BaseModel, File, Path class Output(BaseModel): cropped_table: Path json: Path class Predictor(BasePredictor): def predict(self) -> Output: ... return Output(cropped_table=...,...

"Already running a prediction" When hitting multiple requests

There's a new version https://github.com/replicate/cog/releases/tag/v0.9.0-beta9 that has support for async predictor functions. That might help? cc @technillogue

Error when trying to follow instructions in docs/getting-started.md

Hi @catsby! Thanks for weighing in. @cortfr did bumping the TensorFlow version unblock you here?

`cog login` fails silently if `pass` is uninitialized

Thanks for taking the time to write this up!

`cog login` fails silently if `pass` is uninitialized

Not sure where this would go in the docs, to be honest. We don't really have a troubleshooting doc. Maybe just having this as an issue is a good start,...

typo in description

cc @simonw 👍🏼

Updating UI should be very Thoughtfully Considered

Thanks for taking the time to write this. A few questions: > The default UX is nothing, it just fires an event By this do you mean nothing happens unless...

Updating UI should be very Thoughtfully Considered

@MarshallOfSound are you saying that a successfully downloaded update will be applied automatically no matter what when the app restarts? The documentation on this is sparse, so whatever mysteries we...