search

fal-ai / fal

⚡ Fastest way to serve open source ML models to millions
https://fal.ai/docs
Apache License 2.0
549 stars 49 forks source link

fix(ux): print user messages as is #156

Closed efiop closed 7 months ago

efiop commented 7 months ago

It is unexpected that we print all of them to stdout and also prepend a bunch of logging information. This should be seamless for the user most of the time.

Before:

Screenshot 2024-04-03 at 18 58 49

After:

Screenshot 2024-04-03 at 18 59 14

Same for fal fn run.

linear[bot] commented 7 months ago

FEA-2224 improve remote log printing in fal cli

chamini2 commented 7 months ago

only thing is: we are no longer sure when a print comes from the cloud just by looking at it

efiop commented 7 months ago

@chamini2 Oops, sorry, I think I merged and didn't notice your last comment.

I think that's normal for normal operations and that's what the user expects, using our wrapper shouldn't alter output most of the time. But definitely might want a bit more in debugging mode, I'm working on that.