Jun 13, 2023 · Jun 13, 2023 · Jun 13, 2023 · Jun 13, 2023 · Jun 13, 2023 · Jun 13, 2023
diff --git a/pgml-extension/Dockerfile b/pgml-extension/Dockerfile
 COPY --chown=postgres:postgres . /app
 WORKDIR /app
 RUN pip3 install -r requirements.txt
 RUN pip3 install -r requirements-xformers.txt --no-dependencies

 # Listen on 0.0.0.0 and allow 'root' to connect without a password.
 # Please modify for production deployments accordingly.
 RUN cp /app/docker/postgresql.conf /etc/postgresql/14/main/postgresql.conf
diff --git a/pgml-extension/requirements-xformers.txt b/pgml-extension/requirements-xformers.txt
 xformers==0.0.20
diff --git a/pgml-extension/requirements.txt b/pgml-extension/requirements.txt
 torchvision==0.15.2
 tqdm==4.65.0
 transformers==4.29.2
 xformers==0.0.20
 xgboost==1.7.5
 langchain==0.0.180
 einops==0.6.1
Original file line number	Diff line number	Diff line change
Expand Up		@@ -24,6 +24,8 @@ RUN apt-get update && apt-get install -y postgresql-pgml-14
		COPY --chown=postgres:postgres . /app
		WORKDIR /app
		RUN pip3 install -r requirements.txt
		RUN pip3 install -r requirements-xformers.txt --no-dependencies

		# Listen on 0.0.0.0 and allow 'root' to connect without a password.
		# Please modify for production deployments accordingly.
		RUN cp /app/docker/postgresql.conf /etc/postgresql/14/main/postgresql.conf
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -18,7 +18,6 @@ torchaudio==2.0.2
		torchvision==0.15.2
		tqdm==4.65.0
		transformers==4.29.2
		xformers==0.0.20
Copy link Contributor levkkJun 13, 2023 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. We use this file to install dependencies elsewhere, e.g. in our production cloud. Would it be possible to add another file, perhaps`requirements-xformers.txt` to install that special package? Copy link Contributor levkkJun 13, 2023• edited Loading Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. I think you can do: requirements-xformers.txt `xformers==0.0.20 --no-dependencies` which is then installable with `pip install -r requirements-xformers.txt` Copy link Contributor levkkJun 13, 2023 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. I'm afraid that if we put this package only in the dockerfile, we'll forget to add it elsewhere where it's needed. santiatpml reacted with thumbs up emoji Copy link ContributorAuthor santiatpmlJun 13, 2023 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. `--no-dependencies` has to be outside requirements.txt file. `pip install -r requirements-xformers.txt --no-dependencies`
		xgboost==1.7.5
		langchain==0.0.180
		einops==0.6.1