kubeflow / pipelines / 3713

0%

push

travis-ci-com

Add code for python visualization service (#1651)

* Setup initial server with roc_curve visualization

* Created Dockerfile.visualization

* Fixed import issue

* Changed implementation of generate_html_from_notebook to allow template type to be specified

* Added tfdv.py

* Added unit tests for exporter.py

* Deleted __init__.py

* visualizations/ -> visualization/

* Added requirements.txt and updated Dockerfile.visualization to use it

* Updated .travis.yml to run python visualization unit tests

* Fixed travis file path issue

* Continued testing to fix travis test issues

* Removed jupyter from pip3 install

Previously included to ensure python3 kernel was accessible to jupyter_client.

* Updated requirements.txt to included ipykernel

* Removed maxDiff limit for all python tests

* Sorted keys within args dictionary to ensure tests do not fail due to dictionary order

* Created requirements-test.txt

* Added input_path argument support for python service

Also adds check for missing input_path argument and returns 400 error if argument is missing.

* Updated Copyright in Dockerfile.visualization

* Updated snapshot to include all tests

* Added types, additional comments, and TemplateType enum

Also made additional style changes

* Formatted template files

* Addressed most feedback made by @kevinbache

* Revert "Formatted template files"

This reverts commit a7afd7b8a. This was done due to issues faced by the templating engine.

* Fixed comment placement and switched os -> Path

* Changed way exporter is implemented to use importlib

* Reverted to str.format due to python comparability issue

Python 3.6 introduced support for f-stringsl, this results in the tests failing when run in a python 3.5 environment

* Added unit tests for tornado web server

* Added license script for open source compliance

* Added line between file comment and license to match exporter.py

* Updated server structure

* Created Exporter... (continued)

1572 of 1763 branches covered (89.17%)

Branch coverage included in aggregate %.

2678 of 2817 relevant lines covered (95.07%)

61.44 hits per line