Update API documentation for daily papers #2065
Conversation
Add params documentation for /api/daily_papers
docs/hub/api.md
Outdated
|
|
||
| Other keywords that are supported: | ||
|
|
||
| 1. `limit` - the number of pages returned |
There was a problem hiding this comment.
as in length of the paper or number of papers returned?
There was a problem hiding this comment.
Oops, yeah that's what it should say.
docs/hub/api.md
Outdated
| 1. `limit` - the number of pages returned | ||
| 2. `p` - the page number, for paging through results | ||
| 3. `submitter` - to filter by summitter | ||
| 4. `date`, `week`, `month` - for filtering by date range |
There was a problem hiding this comment.
| 4. `date`, `week`, `month` - for filtering by date range | |
| 4. `date`, `week`, `month` - for filtering by date range |
of the paper release dates?
There was a problem hiding this comment.
I didn't create the API, and it's not clear to me from the openapi docs. It could be either the paper release date, or the day it was featured on Daily Papers.
There was a problem hiding this comment.
According to DeepWiki
The date parameter refers to when the paper was included in the daily papers on the Hub, not when it was written or originally submitted to arXiv. hf_api.py:1527-1528
This is confirmed by the PaperInfo dataclass, which has two distinct date fields:
published_at: The date the paper was published (on arXiv) hf_api.py:1513-1514
submitted_at: The date the paper appeared in daily papers on the Hub hf_api.py:1527-1528|
I will update it according to that.
docs/hub/api.md
Outdated
| 2. `p` - the page number, for paging through results | ||
| 3. `submitter` - to filter by summitter | ||
| 4. `date`, `week`, `month` - for filtering by date range | ||
| 5. `sort` - to specify how results are sorted. |
There was a problem hiding this comment.
how can they be sorted? by date or?
There was a problem hiding this comment.
From the openapi docs:
sortCopy link to sort
enum
default:
"publishedAt"
publishedAt
trending
docs/hub/api.md
Outdated
|
|
||
| Search the daily papers by keyword: | ||
|
|
||
| 1. `q` - the keyword to search |
There was a problem hiding this comment.
this seems standalone so no need to number I think
Co-authored-by: Merve Noyan <merve@huggingface.co>
Clarify the default sorting behavior and options for the API.
Clarified filtering parameters for date and sorting in API documentation.
merveenoyan
left a comment
merveenoyan
left a comment
There was a problem hiding this comment.
lgtm but would like @AK391 to review too just in case
Co-authored-by: Merve Noyan <merve@huggingface.co>
Co-authored-by: Merve Noyan <merve@huggingface.co>
2 participants
This PR documents the parameters for the
/api/daily_papers.This is just a first quick pass. Let me know if you'll take a PR to improve these docs, and any clean-up or edits you'd like me to do.
Thanks @AK391 for pointing me to these docs.