-
Notifications
You must be signed in to change notification settings - Fork 944
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
add a basic bfcl command-line interface #621
base: main
Are you sure you want to change the base?
Conversation
Hi @mattf, Thank you so much for your PR and welcome! I really appreciate your contribution – this feature has been on our TODO list for a while, and it’s great to see it implemented. I noticed a few TODOs left in the code. I'll take care of finishing those up and handle any merge conflicts. After that, we’ll be ready to move forward! |
@HuanzhiMao i'm glad you like it. i've a few more commands i'll push up. |
Perfect. |
my plan was to put a simple cli around the runner / evaluator / model definition code and then propose refactoring changes to make the cli simpler. i've found the cli helpful for my runs, which means it's only had one user. |
I agree. CLI entries will be easier than |
add a simple cli wrapping openfunctions_evaluation.py (
bfcl run
) and eval_runner.py (bfcl evaluate
).