Differential D19273

Make Jupyter notebooks use the fast builtin Python highlighter
ClosedPublic
Actions

Authored by epriestley on Mar 30 2018, 4:48 PM.

Tags

None

Referenced Files

	F15448137: D19273.diff
	Fri, Mar 28, 3:18 AM

	F15442323: D19273.id46136.diff
	Wed, Mar 26, 10:46 PM

	F15441511: D19273.id46136.diff
	Wed, Mar 26, 6:33 PM

	F15441329: D19273.id46134.diff
	Wed, Mar 26, 5:36 PM

	F15439490: D19273.id46134.diff
	Wed, Mar 26, 7:47 AM

	F15436180: D19273.id.diff
	Tue, Mar 25, 12:24 PM

	F15426332: D19273.diff
	Sun, Mar 23, 8:02 AM

	F15425664: D19273.diff
	Sun, Mar 23, 5:20 AM

View All Files

Subscribers

None

Details

Reviewers

mydeveloperday

Maniphest Tasks

T13105: Plans: Rich presentation and diff rendering pipelines for various file types

Commits

rPccbc8a430f62: Make Jupyter notebooks use the fast builtin Python highlighter

Summary

Ref T13105. This is silly, but "py" and "python" end up in different places today, and "py" is ~100x faster than "python".

See also T3626 for longer-term plans on this.

Test Plan

Reloaded a Jupyter notebook, saw it render almost instantly instead of taking a few seconds.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 30 2018, 4:48 PM

Harbormaster completed remote builds in B19955: Diff 46134.Mar 30 2018, 4:49 PM

epriestley requested review of this revision.Mar 30 2018, 4:49 PM

mydeveloperday accepted this revision.Mar 30 2018, 5:19 PM

This revision is now accepted and ready to land.Mar 30 2018, 5:19 PM

epriestley added a child revision: D19274: Support text encoding and syntax highlighting options in document rendering.Mar 30 2018, 6:15 PM

Closed by commit rPccbc8a430f62: Make Jupyter notebooks use the fast builtin Python highlighter (authored by epriestley). · Explain WhyMar 30 2018, 6:26 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

files/

document/

PhabricatorJupyterDocumentEngine.php

2 lines

Diff 46136

View Options

src/applications/files/document/PhabricatorJupyterDocumentEngine.php

Make Jupyter notebooks use the fast builtin Python highlighterClosedPublicActions