Page MenuHomePhabricator

In Jupyter notebooks, read strings stored in the raw as either "string" or "list<string>" more consistently
ClosedPublic

Authored by epriestley on Aug 5 2020, 7:19 PM.
Tags
None
Referenced Files
F19014276: D21434.id.diff
Sat, Nov 22, 7:19 PM
F19004885: D21434.diff
Fri, Nov 21, 1:01 PM
F18829828: D21434.id51004.diff
Oct 25 2025, 1:27 AM
F18814271: D21434.id.diff
Oct 20 2025, 10:47 PM
F18809382: D21434.diff
Oct 19 2025, 1:29 PM
F18804424: D21434.id51005.diff
Oct 18 2025, 8:13 AM
F18797359: D21434.diff
Oct 17 2025, 4:58 AM
F18672812: D21434.id.diff
Sep 25 2025, 8:18 AM
Subscribers
None

Details

Summary

Ref PHI1835. Generally, Jupyter notebooks in the wild may store source and markdown content as either a single string or a list of strings.

Make the renderer read these formats more consistently. In particular, this fixes rendering of code blocks stored as a single string.

This also fixes an issue where cell labels were double-rendered in diff views.

Test Plan

Created a notebook with a code block represented on disk as a single string, rendered a diff from it.

Screen Shot 2020-08-05 at 12.18.44 PM.png (149×1 px, 48 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Aug 5 2020, 7:26 PM
This revision was automatically updated to reflect the committed changes.