-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MDS-6260] Added support to permit condition extraction pipeline to create identified reports #3339
Conversation
Quality Gate passed for 'bcgov-sonarcloud_mds_permits'Issues Measures |
Quality Gate passed for 'bcgov-sonarcloud_mds_core-api'Issues Measures |
@@ -17,7 +17,7 @@ ELASTICSEARCH_PASSWORD=elastic | |||
|
|||
AZURE_API_KEY= | |||
AZURE_API_VERSION=2024-02-01 | |||
AZURE_DEPLOYMENT_NAME=mds-permits-turbo | |||
AZURE_DEPLOYMENT_NAME=gpt-4o |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😮
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Looks good to me!
Objective
MDS-6260
This PR adds support for automatically creating Report Definitions for permit conditions OpenAI flags as needing a report as part of the Permit indexing pipeline. In addition, it contains a couple of fixes and quality of life improvements
Note: This PR only contains the backend part. Putting the frontend changes in a separate PR as we're a few people in there at the moment.
Report Definition Creation
MineReportPermitRequirement
s for conditions that are flagged as needing a report (meta dict -> questions).meta
column : Contains any information extracted about a condition from OpenAI and Azure Document intelligence, including answers to the questions we ask (does the condition require a report, what''s the due date), and things like the bounding box of the condition in the original PDFPermit Condition Extraction Changes
Auto-reload celery worker
Changed core_api_celery to run using watchdog in development to enable hot-reloading. Previously core celery needed a restart to see any changes