Make sdp an optional dependency for docker install

Signed-off-by: Igor Gitman <[email protected]>

pyproject.toml

@@ -20,19 +20,4 @@ py_version = 310  # python 3.10 as a target version
 
 [tool.black]
 line_length = 119
-skip_string_normalization = true
-
-[project]
-name = "nemo_skills"
-description = "NeMo Skills - a project to improve skills of LLMs"
-dynamic = ["version","dependencies"]
-requires-python = ">=3.10"
-readme = "README.md"
-license = {file = "LICENSE"}
-
-[tool.setuptools.packages.find]
-include = ["nemo_skills*"]
-
-[tool.setuptools.dynamic]
-version = {attr = "nemo_skills.__version__"}
-dependencies = {file = ["requirements.txt"]}
+skip_string_normalization = true

requirements.txt

@@ -6,6 +6,5 @@ openai
 pyyaml
 rank_bm25
 requests
-sdp @ git+https://github.com/NVIDIA/NeMo-speech-data-processor
 sympy
 tqdm

setup.py

@@ -22,6 +22,8 @@ def parse_requirements(filename):
 
 # Read the requirements from the requirements.txt file
 requirements = parse_requirements('requirements.txt')
+# sdp is not needed in docker containers, so not adding it to requirements.txt
+requirements.append("sdp @ git+https://github.com/NVIDIA/NeMo-speech-data-processor")
 
 setup(
     name="nemo_skills",