Matthew Seal

Do we have consensus on changing to `extra_engine_arg=dict(...)` pattern with the safe_kwargs pattern in the PR as a warning when it is going to be an issue? I generally like...

So think of papermill's kernel assignment as an override tool. It's usually not used and one relies on the notebook's metadata to make a decision. You're right that the notebook...

Since there's no server managing kernel lifecycles you'd need some extended architecture to lean against to launch and reap kernels. To plug into that architecture you'd need to [register a...

That is interesting, I'll have to play with that when I have a spare weekend. This could possibly fit the secondary goals for scheduling notebooks in the nteract roadmap.

The function `_get_notebook_outputs` does this. We should probably expose it on the notebook object there. I think we might be moving a lot of this logic out of papermill and...

Hi and welcome to the :tada: @cmenguy ! I think there's definitely room in papermill for processing zepplin notebooks. As M mentioned, it definitely operates in a different format than...

If it help, I think the intention here was to actually capture the execution_reply message because the outputs are the source of truth for if there was an execution error....

Dev build fixes the toggle, though I still think a longer default height before snapping to a fixed height would be a better experience. Classic UI actually does this pretty...

True, but in classic it scrolls almost a whole screen height before it hits a threshold and then it snaps back to 336px once it goes past some upper bound...