llm-foundry icon indicating copy to clipboard operation
llm-foundry copied to clipboard

Unify export script arguments after next version release

Open codestar12 opened this issue 2 years ago • 0 comments

🚀 Feature Request

unify export script arguments after next version of LLM-Foundry to improve UX, but not immediately so we don't break people's workflows.

Motivation

inference scripts have nearly identical arguments with different naming conventions (e.g)

convert_hf_mpt_to_ft.py

  --name_or_dir NAME_OR_DIR, -i NAME_OR_DIR
                        HF hub Model name (e.g., mosaicml/mpt-7b) or local dir path to load checkpoint from

and convert_hf_to_onnx.py

  --pretrained_model_name_or_path PRETRAINED_MODEL_NAME_OR_PATH

[Optional] Implementation

check for aligned arguments and pick a naming convention.

codestar12 avatar Jun 29 '23 16:06 codestar12