2738
78%
master: 78%

Ran 15 Nov 2017 11:26PM UTC

Jobs 1

Files 99

Run time 6s

Badge

Embed ▾

pending completion

Build # 2738

Build Type

push

travis-ci

Committed by

qimingj

Commit Message

Fix a performance issue that it loads require.js in each cell output unnecessarily.

I believe the glocal require js config for each cell was added because we now use an iframe for each output cell in experimental UI.

The issue is worth fixing because:

1. require js appends a unique query string parameter to require.js, which makes the file non-cachable, so each output cell needs to download the file (84KB). If the notebook includes 20+ cells (which is very common), it takes long time to load (15s+ difference on my desktop)

2. Each cell output in the notebook includes a require js config section, which makes the notebook size unnecessarily larger.

The fix is to remove the global require js config for each cell output, and add it to only the cell output that need it.

Run Details

5347 of 6884 relevant lines covered (77.67%)

0.78 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
4	2738.4 (TOX_ENV=coveralls)	15 Nov 2017 11:26PM UTC	99	77.67	Travis Job 2738.4

Source Files on build 2738

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

googledatalab / pydatalab / 2738
78%
master: 78%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2738

googledatalab / pydatalab / 2738 78% master: 78%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2738

googledatalab / pydatalab / 2738
78%
master: 78%

README BADGES
x