Merged PR 422753: python documentation fixups

python documentation fixups from #12 and !422716
microsoft · Sep 2, 2020 · 7876605 · 7876605
1 parent 11cbc25
commit 7876605
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 5 deletions.
diff --git a/documentation/01-Prerequisites.md b/documentation/01-Prerequisites.md
@@ -149,21 +149,29 @@ Be sure to include support for .Net Core, C++, CMake
 
 ### Linux Python Install
 
-1. Install Python 3.x
+1. Install Python 3.7
 
  ```sh
- sudo apt -y install python3 python3-pip
+ # We need to add a special apt repository for Python 3.7 support:
+ sudo apt-get -y install software-properties-common apt-transport-https
+ sudo add-apt-repository -y ppa:deadsnakes/ppa
+ sudo apt-get update
+ sudo apt-get -y install python3.7
  ```
 
 2. Install MLOS Python dependencies:
 
  ```sh
  # Also add some dependencies needed by some of the pip modules
- sudo apt -y install build-essential libfreetype-dev unixodbc-dev
+ sudo apt-get -y install python3-pip python3.7-dev \
+ build-essential libfreetype-dev unixodbc-dev
  ```
 
  ```sh
- pip3 install -r source/Mlos.Python/requirements.txt
+ python3.7 -m pip install --upgrade pip
+ python3.7 -m pip install setuptools
+
+ python3.7 -m pip install -r source/Mlos.Python/requirements.txt
  ```
 
 ### Windows Python Install

diff --git a/documentation/CodingStandard.md b/documentation/CodingStandard.md
@@ -28,7 +28,7 @@ We use [`pylint`](https://pypi.org/project/pylint/) for Python code to mostly fo
 
 To run it locally, issues the following commands:
 
-1. One time instal of the `pylint` tool:
+1. One time install of the `pylint` tool:
 
  ```shell
  pip install pylint