Summary:

Added Jupyter Notebooks rendered view.

Review Request #10761 — Created Oct. 20, 2019 and updated Nov. 8, 2019, 10:38 p.m.

Information
Owner:	amalik2
Repository:	Review Board
Branch:	master
Bugs:
Depends On:
Reviewers
Groups:	reviewboard, students
People:

Description

There is also support for a difference viewer that highlights cells in
the notebook that were modified/added/deleted.

This was used as a reference for the format of .ipynb files:
https://github.com/jupyter/nbformat/blob/master/nbformat/v4/nbformat.v4.schema.json

Testing Done

1) I ran unit tests with Python2.7 and Python3.6
2) I tested viewing a file normally with both Python2.7 and Python3.6
3) I tested viewing revisions of a file in the difference viewer with both
Python2.7 and Python 3.6

Commits

Summary	Author
Added Jupyter Notebooks rendered view.	amalik2

Screenshots

Files

Issues

Description	From	Last Updated
E502 the backslash is redundant between brackets	reviewbot	Oct. 20, 2019, 8:14 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 20, 2019, 8:14 p.m.
E502 the backslash is redundant between brackets	reviewbot	Oct. 20, 2019, 8:14 p.m.
E502 the backslash is redundant between brackets	reviewbot	Oct. 20, 2019, 8:14 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 20, 2019, 8:14 p.m.
E502 the backslash is redundant between brackets	reviewbot	Oct. 20, 2019, 8:14 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 20, 2019, 8:14 p.m.
E502 the backslash is redundant between brackets	reviewbot	Oct. 20, 2019, 8:14 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 20, 2019, 8:14 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 20, 2019, 8:14 p.m.
E502 the backslash is redundant between brackets	reviewbot	Oct. 20, 2019, 8:14 p.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 20, 2019, 8:14 p.m.
F401 'django.utils.encoding.force_text' imported but unused	reviewbot	Oct. 20, 2019, 8:14 p.m.
E203 whitespace before ':'	reviewbot	Oct. 20, 2019, 8:14 p.m.

flake8 failed.

JSHint passed.

flake8

reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E502 the backslash is redundant between brackets
```
reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E128 continuation line under-indented for visual indent
```
reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E502 the backslash is redundant between brackets
```
reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E502 the backslash is redundant between brackets
```
reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E128 continuation line under-indented for visual indent
```
reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E502 the backslash is redundant between brackets
```
reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E128 continuation line under-indented for visual indent
```
reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E502 the backslash is redundant between brackets
```
reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E128 continuation line under-indented for visual indent
```
reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E128 continuation line under-indented for visual indent
```
reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E502 the backslash is redundant between brackets
```
reviewboard/reviews/tests/test_jupyter_review_ui.py (Diff revision 1)
Show all issues
```
E128 continuation line under-indented for visual indent
```

reviewboard/reviews/ui/jupyterui.py (Diff revision 1)

Show all issues

F401 'django.utils.encoding.force_text' imported but unused

reviewboard/reviews/ui/jupyterui.py (Diff revision 1)
Show all issues
```
E203 whitespace before ':'
```

Commits:

	Summary	Author
-	Added Jupyter Notebooks rendered view.	amalik2
+	Added Jupyter Notebooks rendered view.	amalik2
+	fixed errors	amalik2

Diff:

Revision 2 (+998 -8)

Show changes

	reviewboard/staticbundles.py
	reviewboard/attachments/mimetypes.py
	reviewboard/reviews/tests/test_jupyter_review_ui.py
	reviewboard/reviews/ui/__init__.py
	reviewboard/reviews/ui/jupyterui.py
	reviewboard/static/rb/css/pages/text-review-ui.less
	reviewboard/static/rb/js/views/jupyterReviewableView.es6.js

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Commits:

	Summary	Author
-	Added Jupyter Notebooks rendered view.	amalik2
-	fixed errors	amalik2
+	Added Jupyter Notebooks rendered view.	amalik2

Diff:

Revision 3 (+1090)

Show changes

	reviewboard/staticbundles.py
	reviewboard/attachments/mimetypes.py
	reviewboard/reviews/tests/test_jupyter_review_ui.py
	reviewboard/reviews/ui/__init__.py
	reviewboard/reviews/ui/jupyterui.py
	reviewboard/static/rb/css/pages/text-review-ui.less
	reviewboard/static/rb/js/views/jupyterReviewableView.es6.js

Added Files:

Checks run (2 succeeded)

flake8 passed.

JSHint passed.