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

Hugging Face API and Unique LLM for Subordinates #221

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

Conversation

neb6dav
Copy link

@neb6dav neb6dav commented Oct 27, 2024

This is my first official Pull Request ever, so I apologize if I'm doing it wrong. I submitted one accidentally to the main branch, as per Eclyp50's request. I then closed it according to Terminally Lazy and submitted it to the Development branch, but in the meantime, I had worked on the Unique LLM per Subordinate problem and so now it is submitting them both. I'm not sure yet how to submit them individually.

The HuggingFace API seems fairly clean and straightforward. Here are my notes on those updates:

Updates to initialize, env, and models to allow for use of HuggingFace API calls. Received Temperature errors if left at 0 when trying to use HuggingFace models, once set to something greater than 0, the errors went away. First time using a HuggingFace API, warning messages were received saying HuggingFace token being saved, on using it a second time in a fresh session no warning messages were received.

The Unique LLMs per Subordinate was much harder for me and I relied heavily on Claude 3.5 Sonnet to re-write a large part of Agent.py to get it working. I hope it's not too bad. I have no background in coding, other than a couple of classes in python from a few years ago. The Unique LLMs required editing agent.py, initialize.py, and call_subordinate.py. One issue I have with this is it currently requires a more "hard-coded" subordinate role, I'd like it to be able to have more freedom in role creation. But, I will keep testing.

Added Hugging Face model
Added Hugging Face API key
Added import HuggingFaceEndPoint from langchain_huggingface also added Hugging Face models. Temperature has to be positive.
Updated to accommodate allowing a0 to generate subordinate roles and assign them with unique chat_model and utility_model, but not Embedding models.
Updated to allow for a0 to create subordinates with "defined" roles, which include unique chat_model and utility_model.
Updated tool so that tool is able to utilize unique chat_model and utility_model.
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