License text issue in image specific model repos 85/6285/2 master
authorEric Z <ezavesky@research.att.com>
Wed, 15 Jan 2020 21:08:01 +0000 (15:08 -0600)
committerEric Z <ezavesky@research.att.com>
Wed, 15 Jan 2020 21:08:01 +0000 (15:08 -0600)
- fix license issue by image update, add proper credit
- remove tox testing in legacy environment
- add existing image for test in webdemo

Issue-ID: ACUMOS-3915
Signed-off-by: Eric Z <ezavesky@research.att.com>
Change-Id: I5e5b58886dabde501a1433ee4349d3da1e3f46c3

docs/catalog_image_detect.png
docs/face-privacy-filter.rst
docs/tutorials/demonstration.rst
tox.ini
web_demo/demo-framework.js
web_demo/face-privacy.js
web_demo/images/face_glasses.jpg [new file with mode: 0644]

index 1f88764..c8cc38d 100644 (file)
Binary files a/docs/catalog_image_detect.png and b/docs/catalog_image_detect.png differ
index cd49b1f..59ec45d 100644 (file)
@@ -192,7 +192,7 @@ detected image regions.
 Metadata Examples
 =================
 
--  example detect catalog image - `url source <https://flic.kr/p/xqw25C>`__
+-  example detect catalog image - `url source <https://www.pexels.com/photo/close-up-photography-of-man-wearing-sunglasses-1212984/>`__
 -  example blur catalog image - `url source <https://flic.kr/p/bEgYbs>`__
 
 
index 8fc405d..04a76da 100644 (file)
@@ -28,10 +28,6 @@ Web Demo
 This web page sample allows the user to submit an image to a face
 detection and a face pixelation service in serial progression.
 
-* ***Image Copyrights May Apply*** - the included sample videos may
-carry additional copyright restrictions and are not meant for public resale or
-consumption.
-
 Browser Interaction
 ===================
 
@@ -110,12 +106,28 @@ model and this web page are using the same ``.proto`` file.
 
       ::
 
-          urlDefault: "http://localhost:8884/transform",
+          urlDefault: "http://localhost:8884/model/methods/detect",
 
 -  view the page ``face-privacy.html`` in a Crome or Firefox browser
 -  you can switch between a few sample images or upload your own by
    clicking on the buttons below the main image window
 
+Compatibility
+-------------
+If you want to run against the Boreas model runner you must set the URL to end
+with /model/methods/detect. The key is that in Boreas release the model runner
+api has changed. You can see the swagger for the model if you visit the source
+model's swagger.
+
+If you have a model that was onboarded using Athena release you do not need
+the /model/methods check. Use /detect instead. Another change that was made
+in Boreas model runner was that you must send the http headers before the
+default was protobuf. These headers will change automatically when use use
+/model/methods/detect as the URL path.
+
+    - Content-type: application/vnd.google.protobuf
+    - Accept: application/vnd.google.protobuf
+
 
 Special decoding example
 ------------------------
diff --git a/tox.ini b/tox.ini
index ebceacd..a797516 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -1,6 +1,6 @@
 [tox]
 skipsdist = True
-envlist = py35,py36
+envlist = py36
 
 [testenv]
 deps = -r{toxinidir}/testing/tox-requirements.txt
index 2997f53..d0be813 100644 (file)
@@ -63,7 +63,8 @@ function demo_init(objSetting) {
         documentTitle: "Protobuf Demo",
         mediaList: [],        //relative URLs of media files
         protoList: [],        //relative URLs of proto files to include
-        domHeaders: { "Content-type": "text/plain;charset=UTF-8" },   //defaults for headers
+        domHeadersV1: { "Content-type": "text/plain;charset=UTF-8" },   //defaults for headers for model runner without path /model/methods/ context
+        domHeadersV2: { "Content-type": "application/vnd.google.protobuf", Accept: 'application/vnd.google.protobuf' },   //defaults for headers
         // TODO: should be binary ideally, domHeaders: { "Content-type": "application/octet-stream;charset=UTF-8" },   //defaults for headers
 
         // Objects from DOM elements
@@ -419,8 +420,12 @@ function doPostImage(srcCanvas, dstDiv, dstImg, imgPlaceholder) {
     //console.log("[doPostImage]: Selected method ... '"+typeInput+"'");
     if (hd.protoKeys) {     //valid protobuf type?
         var blob = dataURItoBlob(dataURL, true);
-        domHeaders = $.extend({}, hd.domHeaders);       //rewrite with defaults
-
+        var newServer = $(document.body).data('hdparams')['classificationServer'];
+        if(newServer.indexOf('/model/methods')!== -1){
+            domHeaders = $.extend({}, hd.domHeadersV2);
+        }else{
+            domHeaders = $.extend({}, hd.domHeadersV1);
+        }
         // fields from .proto file at time of writing...
         // message Image {
         //   string mime_type = 1;
index 3eeeb42..5481f07 100644 (file)
@@ -49,7 +49,12 @@ $(document).ready(function() {
             {
                 'img': 'images/face_reunion.jpg',
                 'source': 'https://flic.kr/p/bEgYbs',
-                'name': 'reuninon (flickr)'
+                'name': 'reunion (flickr)'
+            },
+            {
+                'img': 'images/face_glasses.jpg',
+                'source': 'https://www.pexels.com/photo/close-up-photography-of-man-wearing-sunglasses-1212984/',
+                'name': 'glasses (pexels)'
             },
             {
                 'img': 'images/face_family.jpg',
@@ -60,7 +65,7 @@ $(document).ready(function() {
                 'img': 'images/commercial.jpg',
                 'movie': "images/commercial.mp4",
                 'source': 'https://www.youtube.com/watch?v=34KfCNapnUg',
-                'name': 'family (pexels)'
+                'name': 'commercial (pexels)'
             },
             {
                 'img': 'images/face_Schwarzenegger.jpg',
diff --git a/web_demo/images/face_glasses.jpg b/web_demo/images/face_glasses.jpg
new file mode 100644 (file)
index 0000000..8d6e958
Binary files /dev/null and b/web_demo/images/face_glasses.jpg differ