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

Added a Notebook, requirements.txt, and other. #19

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

Conversation

LuthandoMaqondo
Copy link

@LuthandoMaqondo LuthandoMaqondo commented Nov 24, 2024

Repositpory improvements and Investigating hardware requirements:

  1. Added a separate requirement.txt file for ease of installing necessary dependencies.
  2. Updated the README.md file to to clearly tell repo users about the requirements.txt file
  3. Added a nodebook experiments.ipynb with a Google Colab link for ease of testing Hardware requirements for different inference paramters, becuase requirements depend on the configuration arguments at inference.

I hope this helps other to easily play around with the model and helps in the speed of doing independe evaluations.

It would be great if we could get training/finetuning scripts becuase on my side I have a tone of AWS and Azure credits to work with.

@LuthandoMaqondo LuthandoMaqondo changed the title Luthando contribution Added a Notebook, requirements.txt, and other. Nov 25, 2024
@yoavhacohen yoavhacohen requested a review from SapirW December 7, 2024 19:54
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