ploomber-engine icon indicating copy to clipboard operation
ploomber-engine copied to clipboard

Fix stdout and stderr stream; IO()

Open mehtamohit013 opened this issue 2 years ago • 4 comments

Currently, when running tqdm and printing inside it, breaks and messes up the output for the user. For now, there is a temporary fix, which adds \n to eliminate this but adds extra new lines. Fix the output and the IO() class in ipython.py

#66 #1080 Summary

mehtamohit013 avatar Mar 18 '23 20:03 mehtamohit013

I believe ploomber itself doesn't use ploomber-engine and we'll probably need to change this to solve #1080

idomic avatar Mar 20 '23 21:03 idomic

I believe ploomber itself doesn't use ploomber-engine and we'll probably need to change this to solve https://github.com/ploomber/ploomber/issues/1080

it doesn't use it by default but it can be enabled

edublancas avatar Mar 20 '23 21:03 edublancas

Yeah, I think if we want to solve 1080 we have to right?

idomic avatar Mar 20 '23 21:03 idomic

Yeah, I think if we want to solve 1080 we have to right?

I quickly looked at the source code. It is possible to use ploomber-engine with ploomber but we need to document it more clearly:

https://github.com/ploomber/ploomber/blob/2a6287f1beea9f51ecbe59ddebac266cef6c77e8/src/ploomber/tasks/notebook.py#LL767C5-L767C5

Currently, if you set the engine_name to debug or debuglater, it'll use ploomber-engine because the original motivation for ploomber-engine was to enable debugging.

With the new features we've added, there are now more reasons for users to switch to ploomber-engine; so I think we should allow engine_name: ploomber-engine to also switch to ploomber engine for notebook execution and document what are the benefits of using it.

edublancas avatar Mar 20 '23 23:03 edublancas