Skip to content

Refactor: direct API_Request call via SQL, boto3 instead of AWS CLI with updated formattings. - #1037

Open
Mohd-Hassan wants to merge 2 commits into
mainfrom
customer_reviews
Open

Refactor: direct API_Request call via SQL, boto3 instead of AWS CLI with updated formattings.#1037
Mohd-Hassan wants to merge 2 commits into
mainfrom
customer_reviews

Conversation

@Mohd-Hassan

Copy link
Copy Markdown
Collaborator
  • Updated Vantage workflow to invoke API_Request() directly through SQL .
  • Replaced AWS CLI commands with boto3 due to the ongoing issue with CLI commands.
  • Refactored the notebook to align with the latest formatting standards.

@DallasBowden DallasBowden left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I've updated the markdown for dark mode and modified a couple of messages. I still can't get the TDApiClient.API_Request() to work.

@shilpa-nalkande

Copy link
Copy Markdown
Collaborator

Tested with the new litellm changes. Works fine

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants