Skip to content

Upgrade pyo3 to 0.15#650

Closed
messense wants to merge 1 commit intohuggingface:masterfrom
messense:pyo3-0-13
Closed

Upgrade pyo3 to 0.15#650
messense wants to merge 1 commit intohuggingface:masterfrom
messense:pyo3-0-13

Conversation

@messense
Copy link
Contributor

@messense messense commented Mar 10, 2021

This would require dropping Python 3.5 support.

[features]
default = ["pyo3/extension-module"]

[target.x86_64-apple-darwin]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is simply wrong place to put this.

warning: unused manifest key: target.x86_64-apple-darwin.rustflags

@vks vks mentioned this pull request Jun 11, 2021
@n1t0
Copy link
Contributor

n1t0 commented Jul 5, 2021

Thank you very much for your help with this @messense! I didn't have much time lately but I plan to move forward with this next week probably.

@messense messense changed the title Upgrade pyo3 to 0.13 Upgrade pyo3 to 0.14 Jul 22, 2021
@messense messense changed the title Upgrade pyo3 to 0.14 Upgrade pyo3 to 0.15 Nov 7, 2021
pyo3 = "0.12"
numpy = "0.12"
pyo3 = "0.15.0"
numpy = "0.15.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

numpy does not need to move in sync with pyo3; there also does not exist a numpy 0.15...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It needs to. And it's rust-numpy.

Copy link
Contributor

Choose a reason for hiding this comment

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

OK, makes sense, thanks.

@messense
Copy link
Contributor Author

I'm no long interested in pushing this forward given the lack of review for a year now. Feel free to take over.

@messense messense closed this Mar 21, 2022
@h-vetinari
Copy link
Contributor

I'm no long interested in pushing this forward given the lack of review for a year now. Feel free to take over.

Sorry to hear. I ran into needing this while packaging tokenizers for conda-forge (py3.10 builds fail in cross-compilation without pyo3 0.15)

@h-vetinari h-vetinari mentioned this pull request Mar 21, 2022
@Narsil
Copy link
Contributor

Narsil commented Mar 21, 2022

@messense @h-vetinari Sorry it took so long to review, @n1t0 is hard at work at HF, and simply stopped having time maintaining tokenizers.

I am spending some time to maintain it because I love the library (and rust).
Sorry I didn't really look at very old PRs when I "took over".

Feel free to ping regularly if this ever happens again, GH notifications are sometimes not working too well (or dismissed by error).

Thanks @h-vetinari for taking this over !

Cheers !

@h-vetinari
Copy link
Contributor

If @messense is not too demotivated to take this to completion, this would surely go faster than I'd ever pull it off as a pretty complete newbie. No expectation, but just saying. 😅

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