-
Notifications
You must be signed in to change notification settings - Fork 529
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
Openai compatible gauntlet #1017
base: main
Are you sure you want to change the base?
Conversation
…lm-foundry into migrate_subclasses_to_foundry
…lm-foundry into migrate_subclasses_to_foundry
…lm-foundry into migrate_subclasses_to_foundry
…lm-foundry into migrate_subclasses_to_foundry
…foundry into openai_compatible_gauntlet
…foundry into openai_compatible_gauntlet
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.
Looking good, couple questions but are mostly pointing to stuff that looks like WIP work rather than intended to be merged code
early_stopping_criteria: | ||
- "\n\n" | ||
- "Question:" | ||
# - label: mmlu |
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.
Should these be uncommented or deleted?
cd llm-foundry/scripts | ||
composer eval/eval.py /mnt/config/parameters.yaml | ||
|
||
# Mosaic Cloud will use run_name (with a unique suffix) to populate the env var $RUN_NAME | ||
run_name: mpt-eval | ||
name: mpt-eval-logging |
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.
Is this a temporary change or permanent?
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.
goes for whole file I think?
name: api-eval | ||
cluster: r1z1 # replace with your cluster here! | ||
gpu_num: 8 # | ||
gpu_type: a100_80gb # |
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.
Probably temporary?
if fsdp_config and model_cfg.model.get('load_in_8bit', False): | ||
raise ValueError( | ||
'The FSDP config block is not supported when loading ' + | ||
'Hugging Face models in 8bit.') |
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.
This doesn't seem openai gauntlet related but prob should be merged anyway?
name: hotpotqa | ||
context_length: 65536 | ||
section: beginning | ||
split: test |
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.
Probably don't want to delete these do we?
model_name: gpt-3.5-turbo | ||
- | ||
model_name: openai/davinci | ||
model_name: openai/gpt-3.5-turbo-instruct |
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.
del old gpt-3.5-turbo throughout the codebase for instruct?
@@ -114,8 +114,7 @@ | |||
] | |||
|
|||
extra_deps['openai'] = [ | |||
'openai==1.3.8', | |||
'tiktoken==0.4.0', | |||
'openai==1.3.8', 'tiktoken==0.4.0', 'google-generativeai' |
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.
Should google-generativeai be here or it's own extra_deps category?
OpenAI run:
api-eval-Ik2iMA