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

Windows local developer environment support #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

danivpv
Copy link

@danivpv danivpv commented Oct 7, 2024

PR message

Summary

This pull request adds a Dockerfile, docker-compose.yml, and .dockerignore files to minimize developer setup time and standardize the development environment. These changes are essential for ensuring consistency across different operating systems, especially given some library dependencies on Linux OS.

Usage

To test the setup, populate the .env file with GITHUB_TOKEN and FIREWORKS_API_KEY as key=value pairs, then use the following commands:

docker compose build
docker compose up

Motivation

I made these modifications after encountering OS-dependent library issues while working on Windows. Implementing Docker helps to overcome these OS-specific challenges, providing a more uniform development experience.

Inquiry & Suggestions

Before introducing additional features, I believe it's important to standardize the environment. However, I've noticed that there isn't a dedicated development branch or a clear feature roadmap. I'm curious about plans to scale the features and whether you’re open to supporting an open-source collaborative effort. Could we discuss the roadmap? I'm interested in exploring opportunities such as agent ops, integrating OpenAI's advanced voice model, refactoring code, and removing bark models, as these can complicate local development without a GPU.

Question

On the dockerfile some lines are commented since they introduce breaking changes in the file's structure of the gradio interface.

  • Would making the container have admin rights be a security issue?
  • If not do you mind leaving it like this to work on the roadmap and other features?

…d && docker compose up" with github token and fireworks api keys in .env
@danivpv
Copy link
Author

danivpv commented Oct 7, 2024

BTW. In my local env I added .txt file support and removed bark models since I don't have a GPU. Im eager to refactor and improve the prompt structure with agents to have them iteratively refine the conversation transcript of the podcast through a writer and critic framework with autogen or crewai. BUT before pushing these changes to UI, LLMs, etc... I need to know whether you mind the changes or have some of your own.

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