search

alibaba / GraphTranslator

GraphTranslator:Aligning Graph Model to Large Language Model for Open-ended Tasks
BSD 3-Clause "New" or "Revised" License
68 stars 12 forks source link

Remove pip install for huggingface-cli #1

Closed osanseviero closed 5 months ago

osanseviero commented 5 months ago

The CLI tools are installed when you do pip install huggingface_hub. huggingface-cli is not a Hugging Face package

CLAassistant commented 5 months ago

CLA assistant check
All committers have signed the CLA.

xianbaoqian commented 5 months ago

Just to give some background. This is related to https://www.theregister.com/2024/03/28/ai_bots_hallucinate_software_packages/ where AI generated code hallucinate and mistakenly think that huggingface-cli is a pip dependency to install. huggingface-cli is not an official product from HF and may have security risks. The purpose of this PR is to remove the unused dependency only. The rest of the code works fine.