Skip to content

Conversation

@Samoed
Copy link
Contributor

@Samoed Samoed commented Dec 2, 2025

Ref #3502 (review)

Adding parameters to list_daily_papers:

  • week
  • month
  • submitter
  • sort
  • p
  • limit

Copy link
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pretty good PR @Samoed ! Thanks for completing this API. I've left a few comments regarding default values and tests but apart from that I feel we are close to getting this merged.

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@Wauplin
Copy link
Contributor

Wauplin commented Dec 2, 2025

Don't worry about the failing tests, they are not related to your PR 🤗

Copy link
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating! Everything looks good to me. Will merge it once CI is green 🤗

@Samoed
Copy link
Contributor Author

Samoed commented Dec 3, 2025

Run linter

@Wauplin
Copy link
Contributor

Wauplin commented Dec 3, 2025

thanks for the linting, time to merge now :)

@Wauplin Wauplin merged commit 8626c56 into huggingface:main Dec 3, 2025
17 of 18 checks passed
@Samoed Samoed deleted the daily_ppaper_paramters branch December 3, 2025 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants