Skip to content

Use Hecate PHOEBE bulk endpoint#54

Open
RowanErasmus wants to merge 1 commit into
OHDSI:mainfrom
RowanErasmus:hecate-phoebe-bulk-endpoint
Open

Use Hecate PHOEBE bulk endpoint#54
RowanErasmus wants to merge 1 commit into
OHDSI:mainfrom
RowanErasmus:hecate-phoebe-bulk-endpoint

Conversation

@RowanErasmus

Copy link
Copy Markdown

Hey Rich,

The past month I've had over 245.000 calls come in at the /phoebe endpoint (from 40 users only...) and Joel reached out with the question if maybe he could get requests bulked for Phenelope, so I've added a bulk endpoint and though I might as well get you guys in on it also (even though sorting out phoebe locally at this scale makes more sense to me...)

Hope its of some use :-)

Rowan

Switch the hecate_api PHOEBE provider from per-concept GET requests to the new bulk POST endpoint, chunking concept IDs to reduce request volume against Hecate.

Document PHOEBE_BULK_URL, update tests for the live response shape, and fail closed when legacy PHOEBE_URL_TEMPLATE configuration is still present.
@rkboyce

rkboyce commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

@RowanErasmus Nice! Thank you for the PR. I plan to test this out and accept it after I finish the newest enhancements to the R HADES client. I admit to have been one of the culprits hammering Hecate but not every time - Study Agent will use the local Phoebe DB table as you suggest if the following env is set:

export PHOEBE_PROVIDER=db
export OMOP_DB_ENGINE='<sqlalchemy engine url>'
export VOCAB_DATABASE_SCHEMA=vocabulary
export PHOEBE_DB_TABLE=concept_recommended
export VOCAB_CONCEPT_TABLE=concept

I expect to accept this PR next week or sooner. I am traveling so, some delays, and will reach out again if I have any questions

@RowanErasmus

Copy link
Copy Markdown
Author

Thanks Rich, its ok hammering the api btw, makes me happy to receive a gajilion requests and only quiver for a couple of seconds (its not 100% hammer proof, but I feel its good enough ;-))

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.

2 participants