Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge upstream changes #5

Merged
merged 67 commits into from
Aug 16, 2024
Merged

Merge upstream changes #5

merged 67 commits into from
Aug 16, 2024

Conversation

agrenott
Copy link
Owner

No description provided.

lonvia and others added 30 commits July 15, 2023 19:57
Work-around false positive added by pybind/pybind11@f701654 change:
ItemIterator/CollectionIterator ARE copy/move constructible, even if their template
parameter is not.

Enabled for pybind version >= 2.11.
Requests does not throw on HTTP errors, so pyosmium was just consuming
the error message. Fix that. Also add a retry mechanism for
transient errors. The list of transient status codes is the
same as the one that curl uses.
Fix handling of HTTP errors for replication handler
The werkzeug dependency was not documented.
The only other child class WriteHandler uses a different implementation
for apply, so the generic variant is pointless.
That way, it can be used with generic apply functions.
Generalize osmium.apply() to work with an arbitrary number of handlers
…lass

Allow any Python object to be used as a handler object
This means that apply can now be used with an arbitrary number
of handlers.
@agrenott agrenott force-pushed the dev branch 4 times, most recently from 221b81f to a2d9d45 Compare July 31, 2024 06:57
@agrenott agrenott force-pushed the dev branch 17 times, most recently from da99a98 to ffa74a3 Compare August 16, 2024 21:36
@agrenott agrenott merged commit 25fc123 into npyosmium Aug 16, 2024
22 checks passed
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.

4 participants