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

docs: mention PyBuffer #2593

Merged
merged 2 commits into from
May 9, 2023
Merged

docs: mention PyBuffer #2593

merged 2 commits into from
May 9, 2023

Conversation

davidhewitt
Copy link
Member

@davidhewitt davidhewitt commented Aug 25, 2022

Uses PEP 688 types.Buffer to describe PyBuffer<T> in the conversion tables. Will leave as draft until PEP 688 is finalised.

Closes #954

@adamreichold
Copy link
Member

Will leave as draft until PEP 688 is finalised.

The PEP seems to be accepted, so can this just be "undrafted"?

@davidhewitt davidhewitt added the CI-skip-changelog Skip checking changelog entry label May 9, 2023
@davidhewitt
Copy link
Member Author

Good spot! I've rebased and updated this PR to mention collections.abc.Buffer as per the accepted form of the PEP.

Interestingly the PEP now also defines __buffer__ and __release_buffer__ methods. We may want to consider support for them eventually; I'll create a new ticket.

@davidhewitt davidhewitt marked this pull request as ready for review May 9, 2023 19:21
@davidhewitt
Copy link
Member Author

bors r+

bors bot added a commit that referenced this pull request May 9, 2023
2593: docs: mention PyBuffer r=davidhewitt a=davidhewitt

Uses PEP 688 `types.Buffer` to describe `PyBuffer<T>` in the conversion tables. Will leave as draft until PEP 688 is finalised.

Closes #954 

Co-authored-by: David Hewitt <1939362+davidhewitt@users.noreply.github.com>
@adamreichold
Copy link
Member

adamreichold commented May 9, 2023

It never stops... I think we need to pin wasm-bindgen to version 0.2.84 for the MSRV builds to avoid pulling in syn 2.0.

@bors
Copy link
Contributor

bors bot commented May 9, 2023

Canceled.

@adamreichold
Copy link
Member

bors r+

@davidhewitt
Copy link
Member Author

Thanks for fixing MSRV! How would you feel if I started a patch to bump MSRV to 1.56 as per #3113 ? Would mean we would be waiting until mid June to release 0.19 (i.e. around one month from now)

@adamreichold
Copy link
Member

adamreichold commented May 9, 2023

Thanks for fixing MSRV! How would you feel if I started a patch to bump MSRV to 1.56 as per #3113 ? Would mean we would be waiting until mid June to release 0.19 (i.e. around one month from now)

Personally, I would feel better to release the functionality created so far as 0.19 with our current MSRV (because we know it is compatible and already did the work) and then immediately after that go about bumping both Rust and Python version requirements.

I do loathe the MSRV picket line, but I also think it is important work for us maintainers to make our crates available in as many environments as reasonably possible.

@bors
Copy link
Contributor

bors bot commented May 9, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 633b592 into PyO3:main May 9, 2023
@adamreichold
Copy link
Member

(This of course purely psychological, but 0.20 would also look nicer than 0.19 as "break-stuff-that-used-to-work" release.)

@adamreichold
Copy link
Member

@davidhewitt Penny for your thoughts on whether to bump MSRV before or after releasing 0.19?

@davidhewitt
Copy link
Member Author

I think lets release 0.19 first as you suggest. I'll try to write up a list of what I was hoping to get into 0.19 later

@davidhewitt davidhewitt deleted the docs-pybuffer branch May 15, 2023 20:08
@davidhewitt davidhewitt mentioned this pull request May 15, 2023
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-skip-changelog Skip checking changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Guide entry for PyBuffer
2 participants