Skip to content

Conversation

lukasschwab
Copy link
Owner

Description

Downloads may be a common use case for this package, but the current download helpers are clumsy. They expose (necessarily) partial interfaces for filename-formatting and path construction, they obscure the inner HTTP logic (see urlretrieve).

Instead, the caller can download these files given a URL for them, using whatever HTTP and file logic suits the application. We can add a README example that demonstrates a download with the current default behavior that does not use the newly deprecated methods.

The one clear counterargument I can see: if these download endpoints are subject to the same rate-limiting logic as the API, they should be reimplemented (with reference to a Client) rather than deprecated: managing rate limiting per arXiv policy is a responsibility of this client library.

Breaking changes

List any changes that break the API usage supported on master.

None — strictly additive for the time being: adds source_url method.

Checklist

  • (If appropriate) README.md example usage has been updated.

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.

1 participant