make output_timeout configurable

[renefritze]

I've followed the plugin option for cell-timeout with this and used the previously hardcoded value of 5 as the new default. This would close #136.

[vidartf]

For clarity, do you have some examples of when this could be useful? Are you e.g. generating outputs asynchronously?

[renefritze]

We've since moved on from using nbval, but I think the issue that was fixed for me with this commit was with pythreejs visualizations that wouldn't show otherwise. So my guess is yes, that was async related, but I'm not sure and can no longer check.

[Borda]

We've since moved on from using nbval

so what are you using now?

I think the issue that was fixed

I see this argument in pytest help, but setting any walue does not have any effect #151 (comment)

[Borda]

@fangohr @logicabrity @mvousden mind check this PR?

For clarity, do you have some examples of when this could be useful? Are you e.g. generating outputs asynchronously?

@vidartf on the other hand why have an existing argument that does not have any effect? 🐰

turned out that my cell is hanging even with a long timeout, so the issue is probably also elsewhere...

[mvousden]

[...] @mvousden mind check this PR?

I don't think I'm an appropriate developer to review this, as I haven't contributed to this repository, and I don't have understanding of the issue you're trying to fix. @fangohr may be able to recommend someone suitable though!

[renefritze]

We've since moved on from using nbval

so what are you using now?

We don't ship notebooks that need to be hooked into pytest anymore. We've switched to markdown tutorials in sphinx with myst-nb.

[renefritze]

I'm cleaning up stale PRs, so I'll just leave this here in case there's interest in the future. patch.txt