Merge pull request #62 from creativecommons/issue60

trivial updates to PR #61 (addressing issue #60)

Author: TimidRobot

README.md

@@ -39,9 +39,18 @@ modules:
 - macOS:
   1. Install [Homebrew][homebrew]
   2. Install pipenv:
-        ```
+        ```shell
         brew install pipenv
         ```
+  3. Create the Python virtual environment and install prerequisites using
+     `pipenv`:
+        ```shell
+        pipenv sync --dev
+        ```
+  4. Then you can run the static analysis tools:
+        ```shell
+        ./tools.sh
+        ```
 
 [pipenvdocs]: https://pipenv.pypa.io/en/latest/
 [homebrew]: https://brew.sh/
@@ -71,7 +80,8 @@ To successfully run scripts that require client credentials, you will need to fo
 - [Black][black]: the uncompromising Python code formatter
 - [flake8][flake8]: a python tool that glues together pep8, pyflakes, mccabe,
   and third-party plugins to check the style and quality of some python code.
-- [isort][isort]: A Python utility / library to sort imports.
+- [isort][isort]: A Python utility / library to sort imports
+  - (It doesn't import any libraries, it only sorts and formats them.)
 
 [ccospyguide]: https://opensource.creativecommons.org/contributing-code/python-guidelines/
 [black]: https://github.com/psf/black

flickr/data_cleaning.py

@@ -57,7 +57,7 @@ def save_new_data(
 
 
 def main():
-    drop_empty_column("final.csv", "dataset/cleaned_license10.csv")
+    drop_empty_column("hs.csv", "dataset/cleaned_license10.csv")
     drop_duplicate_id(
         "dataset/cleaned_license10.csv", "dataset/cleaned_license10.csv"
     )