Conversation
|
should be merged only after new version of hf cli is deployed |
burtenshaw
left a comment
There was a problem hiding this comment.
Looks good. I would consider making the description a bit more open, but up to you.
| --- | ||
| name: huggingface-papers | ||
| description: Look up and read Hugging Face paper pages in markdown, and use the papers API for structured metadata such as authors, linked models/datasets/spaces, Github repo and project page. Use when the user shares a Hugging Face paper page URL, an arXiv URL or ID, or asks to summarize, explain, or analyze an AI research paper. | ||
| description: Look up, read, search, and list Hugging Face paper pages using the `hf papers` CLI or the papers REST API. Fetch paper content as markdown, get structured metadata (authors, linked models/datasets/spaces, Github repo, project page), search papers by keyword, and browse the daily papers feed. Use when the user shares a Hugging Face paper page URL, an arXiv URL or ID, or asks to summarize, explain, analyze, search, or list AI research papers. |
There was a problem hiding this comment.
I just want to add AI/ML so that the agent is more encouraged to use it without mention of HF papers.
| description: Look up, read, search, and list Hugging Face paper pages using the `hf papers` CLI or the papers REST API. Fetch paper content as markdown, get structured metadata (authors, linked models/datasets/spaces, Github repo, project page), search papers by keyword, and browse the daily papers feed. Use when the user shares a Hugging Face paper page URL, an arXiv URL or ID, or asks to summarize, explain, analyze, search, or list AI research papers. | |
| description: Look up, read, search, and list artificial intelligence and machine learning paper pages on Hugging Face using the `hf papers` CLI. Fetch paper content as markdown, get structured metadata (authors, linked models/datasets/spaces, Github repo, project page), search papers by keyword, and browse the daily papers feed. Use when the user shares a Hugging Face paper page URL, an arXiv URL or ID, or asks to summarize, explain, analyze, search, or list AI research papers. |
hanouticelina
left a comment
There was a problem hiding this comment.
the hf CLI Skill can be sufficient for this no? is there anything missing there that justifies having a dedicated Skill?
btw, we're improving the skill description to improve Skill triggering as well: huggingface/huggingface_hub#3973
| The Hugging Face team has built an easy-to-use API to interact with paper pages. Content of the papers can be fetched as markdown, or structured metadata can be returned such as author names, linked models/datasets/spaces, linked Github repo and project page. | ||
| The Hugging Face team has built an easy-to-use API and CLI to interact with paper pages. Content of the papers can be fetched as markdown, or structured metadata can be returned such as author names, linked models/datasets/spaces, linked Github repo and project page. | ||
|
|
||
| ## When to Use |
There was a problem hiding this comment.
the agent decides whether to load the skill based only on the frontmatter description, so having this "When to Use" section can't influence triggering no?
@hanouticelina I think the answer is that it uses imo, it's ok for skills here to overlap (partially) with the cli skill, as long as they add extra value to the user. @NielsRogge @mishig25 why does the user need this over the cli skill? |
|
It's mostly minor, the main reasons this skill exist are:
|
|
@NielsRogge nice. to me this sounds useful for academics/students and maybe the cli is more for general MLEs/AI folk. One question. Should we be bringing (and reducing maybe) the paper-publisher skill into this one? |
This PR makes the
huggingface-papersskill up-to-date based huggingface/huggingface_hub#3952.cc @mishig25