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
Adding ParallelSkillSet & tests #31
Adding ParallelSkillSet & tests #31
Conversation
- Replaced import statements to remove unused imports. - Removed unused classes and variables. - Updated select_skill_to_improve to use a lower accuracy threshold. - Added documentation for ParallelSkillSet to explain its purpose.
and update predictions
Thank you for the great PR! Especially for tests ;-) |
adala/skills/skillset.py
Outdated
sequentially for complex problem decompositions and causal reasoning. In the most generic | ||
cases, task decomposition can involve a graph-based approach. | ||
|
||
Attributes: | ||
skills (Dict[str, BaseSkill]): A dictionary of skills in the skill set. | ||
""" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you please remove these non-functional changes? it would be simpler to merge conflicts in this case. Thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done!
Implementing
ParallelSkillSet
class - designed to represent a collection of skills that can be developed and applied simultaneously, as opposed to the sequential approach taken by the existingLinearSkillSet
.This pull request includes basic unit tests for the
ParallelSkillSet
class to ensure its functionality aligns with expected behavior.Thanks Adala team for releasing such an exciting new library!!