openvino.genai icon indicating copy to clipboard operation
openvino.genai copied to clipboard

Try to print git commit id

Open wgzintel opened this issue 2 years ago • 3 comments

wgzintel avatar Dec 20 '23 02:12 wgzintel

@wgzintel @peterchen-intel maybe it is also will be good to introduce versioning into llm_bench? I think it may be more robust way for checking code version instead of commit id in subrpocess (version also can include commit id, as example nncf version use that). Current solution may detect wrong repo and commit if user run benchmarking from another directory, but which is also under git

eaidova avatar Jan 09 '24 07:01 eaidova

@eaidova Do you mean define a file like https://github.com/openvinotoolkit/nncf/blob/release_v280/nncf/version.py#L12C1-L12C12 and adopt https://github.com/openvinotoolkit/nncf/blob/release_v280/setup.py#L77 ?

peterchen-intel avatar Jan 09 '24 09:01 peterchen-intel

@peterchen-intel , yes, I think something similar will be fine, we have no setup, so I think we can combine them in one function get_version, and use it for printing llm_bench version

eaidova avatar Jan 09 '24 09:01 eaidova