Merge pull request #143 from Microsoft/user/xianz/winmldb

update readme and document link

Author: kumraj

Tools/WinMLDashboard/README.md

@@ -32,13 +32,21 @@ The **Convert** tab (see snip below) helps convert models from several different
 
 In order to do the conversion, the tool installs a separate Python environment and a set of converter tools. This helps alleviate one of the big pain points of a typical developer - installing the right Python environment and tool chain for conversion and validating the model. 
 
-WinMLTools provides quantization feature to reduce the memory footprint of the model.
+WinMLTools provides [quantization](https://docs.microsoft.com/en-us/windows/ai/convert-model-winmltools#quantize-onnx-model) feature to reduce the memory footprint of the model.
 
-For **quantization**:
-|Type |Usage   |
-|-----|--------|
-|**Type1**|Do not use dequantize_linear option, the converted model can only be run in Win10 released before [Version 1809 (OS build 17763)](https://support.microsoft.com/en-us/help/4464619/windows-10-update-history)|
-|**Type2**|Use dequantize_linear option, the converted model can also be run in Win10 release after Version 1809 (OS build 17763)|
+For **quantization:**
+
+|Type        |Usage  |
+|------------|-------|
+|**Type1**   |Do not use dequantize_linear option, the converted model can only be run in Win10 released before [Version 1809 (OS build 17763)](https://support.microsoft.com/en-us/help/4464619/windows-10-update-history)   |
+|**Type2**   |Use dequantize_linear option, the converted model can also be run in Win10 release after Version 1809 (OS build 17763) |
+
+For **ONNX Version:**
+
+|Version     |For Windows 10 Build|
+|------------|--------------------|
+|1.2         |[1809 (OS build 17763)](https://support.microsoft.com/en-us/help/4464619/windows-10-update-history)|
+|1.3         |after 1809 (OS build 17763)|
 
 <img src="./public/Converter.png" width=800/>
 

Tools/WinMLDashboard/build.js

@@ -5,7 +5,7 @@ resultPromise = electronInstaller.createWindowsInstaller({
     appDirectory: path.join('./release/WinMLDashboard-win32-x64'), 
     authors: 'Microsoft Corporation',
     exe: 'WinMLDashboard.exe',
-    iconUrl: 'https://github.com/Microsoft/Windows-Machine-Learning/blob/RS5/Tools/WinMLDashboard/public/winml_icon.ico',
+    iconUrl: 'https://github.com/Microsoft/Windows-Machine-Learning/blob/master/Tools/WinMLDashboard/public/winml_icon.ico',
     loadingGif: './public/progress_bar.gif',
     noMsi: true,
     outputDirectory: path.join('./installer'), 

Tools/WinMLDashboard/src/native/menu.ts

@@ -22,7 +22,7 @@ export function createMenu(electron: typeof Electron) {
                 {
                     label: 'Documentation',
                     click() {
-                        const tpnUrl = 'https://github.com/Microsoft/Windows-Machine-Learning/blob/RS5/Tools/WinMLDashboard/README.md';
+                        const tpnUrl = 'https://github.com/Microsoft/Windows-Machine-Learning/blob/master/Tools/WinMLDashboard/README.md';
                         require('electron').shell.openExternal(tpnUrl);
                     },
                 },
@@ -39,7 +39,7 @@ export function createMenu(electron: typeof Electron) {
                 {
                     label: 'Third Party Notice',
                     click() {
-                        const tpnUrl = 'https://github.com/Microsoft/Windows-Machine-Learning/blob/RS5/Tools/WinMLDashboard/ThirdPartyNotice.txt';
+                        const tpnUrl = 'https://github.com/Microsoft/Windows-Machine-Learning/blob/master/Tools/WinMLDashboard/ThirdPartyNotice.txt';
                         require('electron').shell.openExternal(tpnUrl);
                     },
                 },

Tools/WinMLDashboard/src/view/convert/View.tsx

@@ -1,4 +1,5 @@
 import * as fs from 'fs';
+import * as path from 'path';
 import * as React from 'react';
 import { connect } from 'react-redux';
 
@@ -9,6 +10,8 @@ import { Spinner } from 'office-ui-fabric-react/lib/Spinner';
 import { TextField } from 'office-ui-fabric-react/lib/TextField';
 import Select from 'react-select';
 
+import { winmlDataFolder } from '../../native/appData';
+
 import Collapsible from '../../components/Collapsible';
 import { setFile, setQuantizationOption, setSaveFileName } from '../../datastore/actionCreators';
 import IState from '../../datastore/state';
@@ -125,7 +128,7 @@ class ConvertView extends React.Component<IComponentProperties, IComponentState>
                 return <Spinner label="Converting..." />;
         }
         this.localPython = this.localPython || getLocalPython();
-        if (!this.localPython) {
+        if (!this.localPython || !fs.existsSync(path.join(winmlDataFolder, '/python/PIResult.txt'))) {
             return this.pythonChooser();
         }
         return this.converterView();
@@ -184,6 +187,10 @@ class ConvertView extends React.Component<IComponentProperties, IComponentState>
                 await pip(['install', '-r', packagedFile('requirements.txt'), '--no-warn-script-location'], this.outputListener);
                 this.setState({ currentStep: Step.Idle });
                 log.info("python environment is installed successfully");
+                fs.writeFile(path.join(winmlDataFolder, '/python/PIResult.txt'), 'Python is installed successfully!', (err) => {
+                    if (err) { throw err; }
+                    log.info('PIResult.txt Saved!');
+                  });
                 pythonDialogOptions.message = 'python environment is installed successfully!'
                 require('electron').remote.dialog.showMessageBox(require('electron').remote.getCurrentWindow(), pythonDialogOptions)
             } catch (error) {
@@ -362,7 +369,7 @@ class ConvertView extends React.Component<IComponentProperties, IComponentState>
         }
         // Convert successfully
         log.info(this.state.source + " is converted successfully.");
-        convertDialogOptions.message = 'convert successfully!'
+        convertDialogOptions.message = 'Converted successfully!'
         require('electron').remote.dialog.showMessageBox(require('electron').remote.getCurrentWindow(), convertDialogOptions)
         this.setState({ currentStep: Step.Idle, source: undefined, console:"Converted successfully!!"});