Skip to content
This repository has been archived by the owner on Nov 1, 2024. It is now read-only.

AudioGen - Implemented Activities Audio Generation [DRAFT PR] #120

Open
wants to merge 58 commits into
base: main
Choose a base branch
from

Conversation

Nate8888
Copy link

@Nate8888 Nate8888 commented Dec 1, 2023

Description

This pull request introduces several new features to the Audiogen tool, aimed at enhancing the audio generation capabilities for our users. The new functionalities allow users to generate audio based on provided activities. The new feature list is as follows:

  • Ability to generate audio based on daily activities from a comma-separated string or a .ics calendar file.
  • Ability to integrate with GPT models to enhance activity descriptions.
  • Ability to enable pseudo-deterministic activity prompts
  • Ability to specify a date or a range of dates to get events from the .ics calendar file.

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Feature/Issue validation/testing

To validate the changes, the script was executed with various options. Here are the validation/testing results:

  • Test A - Single Description, Default Settings

  • Test B - Single Activity list, Default Settings

  • Unit Test / Integration Test (UT/IT) execution results

tests/test_main.py::test_single_description PASSED
tests/test_main.py::test_activity_to_sound PASSED
2 passed in 164.70s (0:02:44)

Checklist:

  • Have you added tests that prove your fix is effective or that this feature works?
  • New and existing unit tests pass locally with these changes?
  • Has code been commented, particularly in hard-to-understand areas?
  • Have you made corresponding changes to the documentation?

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 1, 2023
@Nate8888 Nate8888 marked this pull request as draft December 1, 2023 17:40
@Nate8888 Nate8888 marked this pull request as ready for review December 1, 2023 17:40
@Nate8888
Copy link
Author

Nate8888 commented Dec 1, 2023

Hi, @jfResearchEng! Let me know if there are any changes you would like me to make. I had to use my branch from the previous PR as a base to get all the support features. If it becomes hard to review, we can merge PR 117 and then I can rebase it on main or a different branch so you can only see the appropriate diff content

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants