Removing the notebookDeprecated proposal

The notebookDeprecated api proposal is currently not on track for finalization. It only contains a single property NotebookCellOutput.id, which we believe is no longer useful or needed

We plan on removing this proposal it once all current consumers have migrated off it. Here’s the current list:

• Jupyter @rchiodo @DonJayamanne
• Data Wrangler @pwang347
• Dot Net Interactive @brettfo, @colombod

Please check if your extension is using this api and if you have any concerns migrating off of it

[jrieken]

• id is useful to find ext host output from renderer (command, actions, etc)
• tho id is random and doesn’t fit into model as nicely
• think about exposing more information about cell output to renderer like index of the output, index of cell, uri and version of document

[rchiodo]

I think the assumption that we don’t want metadata in the output could be challenged. We already put metadata in the kernel info. Adding it to the output doesn’t seem bad to me.

And writing metadata need only happen when we update an output. Output without metadata.ids could just skip supporting the plot viewer.

Additionally we might not even need it. You could hash the contents of the image and when clicking the plot viewer or plot save button, pass the hash of the image. The extension side code could just look for this image in the output data.