Skip to content

Commit

Permalink
fix: debug demo (#49)
Browse files Browse the repository at this point in the history
* fix: specify email password

* fix: only load dask when required
  • Loading branch information
mjaquiery authored Jan 20, 2025
1 parent 3fbbc08 commit 8286393
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 5 deletions.
6 changes: 4 additions & 2 deletions .github/workflows/demo.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,14 +19,15 @@ jobs:
runs-on: ubuntu-latest
concurrency: demo-group
env:
DEMO_BACKEND_VERSION: "v2.4.2"
DEMO_BACKEND_VERSION: "v2.5.1-dev"
FRONTEND_URL: "https://galv-demo.fly.dev"
FLY_DEMO_NAME: "galv-demo-backend"
FLY_DEMO_DB_NAME: "galv-demo-db"
FLY_ORG: "oxrse"
FLY_API_TOKEN: ${{ secrets.FLY_API_TOKEN }}
DJANGO_SECRET_KEY: ${{ secrets.DJANGO_DEMO_SECRET_KEY }}
DJANGO_SUPERUSER_PASSWORD: ${{ secrets.DJANGO_DEMO_SUPERUSER_PASSWORD }}
DJANGO_EMAIL_HOST_PASSWORD: ${{ secrets.DJANGO_DEMO_EMAIL_HOST_PASSWORD }}
steps:
- name: Checkout code
uses: actions/checkout@v4
Expand Down Expand Up @@ -85,7 +86,8 @@ jobs:
--config ./fly.demo.toml \
--stage \
DJANGO_SECRET_KEY="$DJANGO_SECRET_KEY" \
DJANGO_SUPERUSER_PASSWORD="$DJANGO_SUPERUSER_PASSWORD"
DJANGO_SUPERUSER_PASSWORD="$DJANGO_SUPERUSER_PASSWORD" \
DJANGO_EMAIL_HOST_PASSWORD="$DJANGO_EMAIL_HOST_PASSWORD"
echo "Deploying app $FLY_DEMO_NAME"
flyctl deploy \
--app $FLY_DEMO_NAME \
Expand Down
5 changes: 3 additions & 2 deletions backend_django/galv/serializers/serializers.py
Original file line number Diff line number Diff line change
Expand Up @@ -78,8 +78,6 @@
from rest_framework import serializers
from knox.models import AuthToken

from galv_harvester.harvest import InternalHarvestProcessor

from .utils import (
CustomPropertiesModelSerializer,
GetOrCreateTextField,
Expand Down Expand Up @@ -2225,6 +2223,9 @@ def create(self, validated_data):
it's less of a headache than trying to create persistent temporary storage somewhere
and police the limits on it.
"""
from galv_harvester.harvest import (
InternalHarvestProcessor,
) # this pulls in a lot, so only import when needed

def df_to_dict(df) -> dict:
return json.loads(df.to_json())
Expand Down
2 changes: 1 addition & 1 deletion docs/tags.json
Original file line number Diff line number Diff line change
@@ -1 +1 @@
["v2.2.0", "v2.3.1", "v2.5.0-dev", "main"]
["v2.2.0", "v2.3.1", "v2.5.1-dev", "main"]

0 comments on commit 8286393

Please sign in to comment.