Skip to content

Commit

Permalink
Force galvani and navani update in lockfile (#602)
Browse files Browse the repository at this point in the history
* Force galvani and navani update in lockfile; important to generate with older Python

* Dial down default `WEB_CONCURRENCY`

* Also skip LFS smudge in CI

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
  • Loading branch information
ml-evs and pre-commit-ci[bot] authored Feb 19, 2024
1 parent e23e694 commit 0807964
Show file tree
Hide file tree
Showing 6 changed files with 624 additions and 602 deletions.
5 changes: 4 additions & 1 deletion .docker/server_dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@
FROM python:3.10 as base
SHELL ["/bin/bash", "--login", "-c"]

# Useful for installing deps from git, preventing big downloads and bandwith quotas
ENV GIT_LFS_SKIP_SMUDGE=1

# Install system dependencies
RUN apt update && apt install -y gnupg curl tree mdbtools && apt clean

Expand Down Expand Up @@ -32,7 +35,7 @@ RUN [ "pipenv", "run", "pip", "install", "gunicorn" ]
COPY ./pydatalab/ ./

# This will define the number of gunicorn workers
ARG WEB_CONCURRENCY=16
ARG WEB_CONCURRENCY=4
ENV WEB_CONCURRENCY=${WEB_CONCURRENCY}

ARG PORT=5001
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@ on:

env:
FORCE_COLOR: true
# Need to set this to avoid pipenv syncing an entire LFS repo and failing with bandwidth quota (see
# https://github.com/the-grey-group/datalab/issues/603)
GIT_LFS_SKIP_SMUDGE: 1

concurrency:
group: ${{ github.head_ref || github.run_id }}
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/ci_release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ env:
PUBLISH_UPDATE_BRANCH: main
GIT_USER_NAME: "Greymon (bot)"
GIT_USER_EMAIL: "[email protected]"
# Need to set this to avoid pipenv syncing an entire LFS repo and failing with bandwidth quota (see
# https://github.com/the-grey-group/datalab/issues/603)
GIT_LFS_SKIP_SMUDGE: 1

jobs:
publish:
Expand Down
2 changes: 1 addition & 1 deletion docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ services:
dockerfile: .docker/server_dockerfile
target: production
args:
- WEB_CONCURRENCY=16
- WEB_CONCURRENCY=4
depends_on:
- database
volumes:
Expand Down
4 changes: 2 additions & 2 deletions pydatalab/Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ pandas = "*"
scipy = "*"
matplotlib = "*"
pymongo = "*"
galvani = "*"
openpyxl = "*"
nmrglue = "*"
python-dotenv = "*"
Expand All @@ -28,7 +27,8 @@ flask-compress = "*"
pillow = "*"
openai = "~= 0.28"
tiktoken = "*"
navani = {git = "git+https://github.com/the-grey-group/[email protected]"}
galvani = {git = "git+https://github.com/echemdata/galvani.git@1fd9f84"}
navani = {git = "git+https://github.com/the-grey-group/[email protected]"}
python-dateutil = "*"
pybaselines = "*"
rosettasciio = "*"
Expand Down
Loading

0 comments on commit 0807964

Please sign in to comment.