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
Unknown Object (File)
Wed, Mar 27, 12:41 PM
Unknown Object (File)
Sun, Mar 10, 7:03 PM
Unknown Object (File)
Thu, Feb 29, 8:20 PM
Unknown Object (File)
Jan 9 2024, 8:03 AM
Unknown Object (File)
Jan 7 2024, 5:04 PM
Unknown Object (File)
Jan 5 2024, 1:18 PM
Unknown Object (File)
Jan 3 2024, 7:31 PM
Unknown Object (File)
Dec 30 2023, 6:12 PM
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
Branch
jupyter1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24821
Build 34244: Run Core Tests
Build 34243: arc lint + arc unit

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.