Skip to content

Commit

Permalink
Merge pull request #80 from FrancisCrickInstitute/NEUT-17
Browse files Browse the repository at this point in the history
specify db driver in connection string (fixes #NEUT-17)
  • Loading branch information
Swarchal authored Sep 5, 2024
2 parents cd6ba2d + 5322eaf commit c208dfc
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions plaque_assay/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,17 +3,15 @@
the titration analysis.
"""


import os
from typing import List

import sqlalchemy
from plaque_assay.experiment import Experiment
from plaque_assay.errors import DatabaseCredentialError

from plaque_assay import ingest, utils
from plaque_assay.db_uploader import AnalysisDatabaseUploader
from plaque_assay import ingest
from plaque_assay import utils
from plaque_assay.errors import DatabaseCredentialError
from plaque_assay.experiment import Experiment


def create_engine(test: bool = True) -> sqlalchemy.engine.base.Engine:
Expand Down Expand Up @@ -49,7 +47,9 @@ def create_engine(test: bool = True) -> sqlalchemy.engine.base.Engine:
"db credentials not found in environent.",
"Need to set NE_USER, NE_HOST_{TEST,PROD}, NE_PASSWORD",
)
engine = sqlalchemy.create_engine(f"mysql://{user}:{password}@{host}/serology")
engine = sqlalchemy.create_engine(
f"mysql+mysqldb://{user}:{password}@{host}/serology"
)
return engine


Expand Down

0 comments on commit c208dfc

Please sign in to comment.