n-riesco / jp-kernel

Generic Node.js kernel for the Jupyter notebook
Other
21 stars 15 forks source link

startupScript fails silently #11

Closed apowers313 closed 1 year ago

apowers313 commented 3 years ago

Startup scripts fail silently if they are broken and logging isn't enabled.

https://github.com/n-riesco/jp-kernel/blob/50dff2adaf69614f74814182805ef30827411017/lib/jp-kernel.js#L401

n-riesco commented 3 years ago

This is by design and it applies to all logging messages. To give you a little bit of background for this decision. A few years back Jupyter frontends would only accept stream messages to report writes to stdout and stderr from a kernel session. Users of some academic libraries (e.g. tensorflow) wanted to see the writes these libraries made directly to stdout and stderr. The response of frontends was to report all writes to stdout and stderr as if they were stream messages; the unfortunate side-effect being the inclusion of kernel logging messages. For that reason I decided to hide logging messages behind a flag.