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

Revamped the project structure – everything's tidier now! #66

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

Conversation

abdalrohman
Copy link

  • Created a new rag_api directory to house all the core components of the project.
  • Inside rag_api, I organized things into a few subdirectories to keep things tidy.

Hopefully, this makes the codebase feel a bit less cluttered and more intuitive as we continue to build on it.

- Grouped packages by categories for better readability.
- Updated package versions to the latest available from PyPI.
- Applied changes to requirements.txt and requirements.lite.txt
- use: `conda env create -f environment.yml`
- Created a new `rag_api` directory to keep related parts of the code together.
- Created subdirectories within `rag_api` to group related modules.

This should make the code easier to understand and work with as the project grows!
@danny-avila
Copy link
Owner

Hey @abdalrohman

Just want to say thank you for your interest in the RAG API in general. I would say you're the # 1 contributor for this.

In the next week or two, I'm really going to sit down with this repo and make optimizations.

This is great, but I'm going to merge some of the other open PRs so there are less merge conflicts all around.

@abdalrohman
Copy link
Author

@danny-avila Thanks, Danny! I'm glad to contribute and excited to see the upcoming optimizations. Looking forward to collaborating more.

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.

3 participants