Update changelog, remove requirements.txt inference logic from models.py, pip install unboxapi for fetal health examples

Author: gustavocidornelas

CHANGELOG.md

@@ -7,7 +7,11 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
-### Fixed
+# Changed
+
+* `requirements_txt_file` no longer optional for model uploads. 
+
+# Fixed
 
 * Added protobuf==3.2.0 to requirements to fix bug with model deployment.
 

examples/tabular-classification/sklearn/fetal-health/fetal-health-sklearn.ipynb

@@ -611,6 +611,22 @@
  "source": [
  "## Unbox part!\n",
  "\n",
+ "### pip installing unboxapi"
+ ]
+ },
+ {
+ "cell_type": "code",
+ "execution_count": null,
+ "metadata": {},
+ "outputs": [],
+ "source": [
+ "!pip install unboxapi"
+ ]
+ },
+ {
+ "cell_type": "markdown",
+ "metadata": {},
+ "source": [
  "### Instantiating the client"
  ]
  },

unboxapi/__init__.py

@@ -208,7 +208,7 @@ def add_model(
  model,
  model_type: ModelType,
  class_names: List[str],
- requirements_txt_file: str = None,
+ requirements_txt_file: str,
  feature_names: List[str] = [],
  categorical_feature_names: List[str] = [],
  train_sample_df: pd.DataFrame = None,

unboxapi/models.py

@@ -113,20 +113,19 @@ def _format_custom_code(custom_model_code: Optional[str]) -> str:
 
 
 def _env_dependencies(
- tmp_dir: str, requirements_txt_file: Optional[str], setup_script: Optional[str]
+ tmp_dir: str, requirements_txt_file: str, setup_script: Optional[str]
 ):
  unbox_req_file = f"{tmp_dir}/requirements.txt"
  env_wrapper_str = ""
- if not requirements_txt_file:
- env_wrapper_str += "@env(infer_pip_packages=True"
- else:
- shutil.copy(requirements_txt_file, unbox_req_file)
- # Add required dependencies
- deps = [f"bentoml=={bentoml.__version__}", "pandas"]
- with open(unbox_req_file, "a") as f:
- f.write("\n")
- [f.write(f"{dep}\n") for dep in deps]
- env_wrapper_str += f"@env(requirements_txt_file='{unbox_req_file}'"
+
+ shutil.copy(requirements_txt_file, unbox_req_file)
+ # Add required dependencies
+ deps = [f"bentoml=={bentoml.__version__}", "pandas"]
+ with open(unbox_req_file, "a") as f:
+ f.write("\n")
+ [f.write(f"{dep}\n") for dep in deps]
+
+ env_wrapper_str += f"@env(requirements_txt_file='{unbox_req_file}'"
 
  # Add a user defined setup script to execute on startup
  if setup_script: