4
78%
master: 78%

Ran 16 Nov 2017 02:03AM UTC

Files 99

Run time 3s

Badge

Embed ▾

Committed 16 Nov 2017 01:43AM UTC coverage: 77.673% (+0.2%) from 77.503%

Job # TOX_ENV=coveralls

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Fix a performance issue that it loads require.js in each cell output unnecessarily. (#615)

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

googledatalab / pydatalab / 2744 / 4
78%
master: 78%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2744.4 (TOX_ENV=coveralls)

googledatalab / pydatalab / 2744 / 4 78% master: 78%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2744.4 (TOX_ENV=coveralls)

googledatalab / pydatalab / 2744 / 4
78%
master: 78%

README BADGES
x