lukasschwab / arxiv.py

Python wrapper for the arXiv API

MIT License

1.07k stars 120 forks source link

Introduce v1.0.0 client: refactor into `Client`, `Search`, `Result` #51

Closed lukasschwab closed 3 years ago

lukasschwab commented 3 years ago

Description

There are a few overarching goals to this change:

Simplify the client interface. The function signature for query includes a great number of options.
Make client more robust, including against brittleness in the underlying API.
Prepare for the eventual release of an arXiv JSON API: complete an abstraction layer over the Atom-specific tooling.

To that end, this rewrite is designed as follows:

Introduces Search and Client classes for managing client behavior and running queries.
- Former query arguments are split between them so that Search doesn't include any values specifying a request strategy.
- Introduces retries in the Search class.
Uses a new Result class in lieu of raw FeedParserDict objects.
Introduces ArxivError, child classes.

Breaking changes

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

This breaks the whole v0.x interface. Certain features have not been reimplemented:

Nongenerator queries. This can be equivalently achieved with list(generator); the nongenerator option was always about backwards-compatibility, which is no longer of concern.
start is no longer exposed externally. This can be re-added as a Search option if there are requests for it.

Development changes

Added pdoc generated documentation to ./docs, which will be hosted using GitHub Pages.
Added linting with flake8; ignoring the line length rules to permit descriptive comments.

Relevant issues

List GitHub issues relevant to this change.

https://github.com/lukasschwab/arxiv.py/issues/43

Checklist

[x] All tests pass: run make test.
[x] All API changes are documented in README.md.
[x] Ready for review.