Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #
🤖 AI-Generated PR Description (Powered by Amazon Bedrock)
Description
This pull request includes changes to multiple files across different directories. The primary changes are:
source/infrastructure/lib/model/model-construct.ts
: This file has been modified, likely to update the infrastructure code related to the model deployment.source/lambda/job/dep/llm_bot_dep-0.1.0-py3-none-any.whl
: There is an error with this file, which appears to be a Python wheel package. This issue needs to be investigated and resolved.source/lambda/job/dep/llm_bot_dep/loaders/pdf.py
: This Python file, responsible for loading PDF files, has been modified.source/model/etl/code/main.py
: Changes have been made to the main Python file responsible for the ETL (Extract, Transform, Load) process of the model.source/model/etl/code/requirements.txt
: The requirements file for the ETL process has been updated, likely to include new dependencies or update existing ones.The motivation behind these changes is to enhance the model deployment infrastructure, improve the PDF loading capabilities, and update the ETL process for the model. No specific dependencies have been mentioned, but the changes to the
requirements.txt
file suggest that new dependencies may be required.Type of change
File Stats Summary
File number involved in this PR: 4, unfold to see the details:
The file changes summary is as follows:
invoke_etl_model
function, includingmodel_provider
,model_id
,api_secret_name
, andapi_url
. It also refactors theprocess_pdf
function to use a dictionary for passing parameters toinvoke_etl_model
, allowing for easier extensibility.