Skip to content
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

fix path to lambda_handler in dockerfile #39

Merged
merged 14 commits into from
Dec 9, 2024
Merged

Conversation

bendsouza2
Copy link
Owner

No description provided.

@bendsouza2
Copy link
Owner Author

  • fix path to lambda_handler in docker file
  • and enchant-devel install to fix enchant lib import
  • aws creds hierarchy - ensure local methods not used when running in aws
  • add event and context args to lambda handler

@bendsouza2
Copy link
Owner Author

Included in this PR:

  • fix path to lambda_handler in docker file
  • and enchant-devel install to fix enchant lib import
  • add spanish dictionary installs to docker image
  • aws creds hierarchy - ensure local authentication not used when running in aws
  • add event and context args to lambda handler
  • add logging and apply logging to all methods
  • add lambda specific requirements
  • add audio_cloud_path attribute to Audio class
  • deprecate echogarden_generate_subtitles method --> use temp srt file generation from VideoGenerator class instead
  • ensure generate_video method writes the temp_audio_file to a /tmp directory when running in lambda to prevent writing file to a read only file system
  • add path to fonts in python code and ensure fonts are installed in docker image
  • closes aws support #17 ----> aws support fully integrated

@bendsouza2 bendsouza2 merged commit 0a1ed07 into main Dec 9, 2024
1 check passed
@bendsouza2 bendsouza2 deleted the bug/docker-lambda-entry branch December 9, 2024 20:35
@bendsouza2 bendsouza2 mentioned this pull request Dec 10, 2024
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.

1 participant