Skip to content

Conversation

@ballPointPenguin
Copy link
Member

No description provided.

@ballPointPenguin ballPointPenguin force-pushed the br/python-better branch 2 times, most recently from d369e6e to 33cd326 Compare October 17, 2025 05:36
"Delphi_PCAConversationConfig": {
"KeySchema": [{"AttributeName": "zid", "KeyType": "HASH"}],
"AttributeDefinitions": [{"AttributeName": "zid", "AttributeType": "S"}],
"ProvisionedThroughput": {"ReadCapacityUnits": 5, "WriteCapacityUnits": 5},
Copy link
Collaborator

@tevko tevko Oct 17, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be NO provisioned throughput on any table we use or create, all tables should rely on "on-demand" capacity, aka "BillingMode": "PAY_PER_REQUEST",

[ ] Audit and fix pytest tests
[ ] Establish delphi tests github workflow
[x] Refactor make docker commands to use docker directly without docker compose
[ ] Confirm or remove `configure_instance.py`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some of these are used in prod application setup process during code deploy phase in AWS

# SENTENCE_TRANSFORMER_MODEL=sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2

# Database configuration
DATABASE_HOST=localhost
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ideally we don't have this at all / all env config is handled at project root level

@@ -119,13 +119,13 @@ services:
image: 050917022930.dkr.ecr.us-east-1.amazonaws.com/polis/delphi:latest
build:
context: ./delphi
container_name: delphi-app
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to make sure this doesn't conflict with prod setup in AWS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants