Code Monkey home page Code Monkey logo

Comments (15)

dcwatson avatar dcwatson commented on June 24, 2024 2

Finally pushed out 0.4.0 to PyPI, including pre-built wheels for Python 3.8-3.11 on all platforms. I still think it's worth splitting into two separate packages (source vs. binary) but it's been long enough...

from deflate.

ThomasWaldmann avatar ThomasWaldmann commented on June 24, 2024

changelog - shall we help? next release is 0.4.0?

from deflate.

ThomasWaldmann avatar ThomasWaldmann commented on June 24, 2024

#29 is a good change and ready to merge into master, should get into next release i guess.

from deflate.

ThomasWaldmann avatar ThomasWaldmann commented on June 24, 2024

looks like @dcwatson is busy right now, so i'll just work on a changelog PR...

see #30.

from deflate.

ThomasWaldmann avatar ThomasWaldmann commented on June 24, 2024

btw, @dcwatson, if you like to increase the "bus factor" a bit, you could add me as addtl. maintainer (or owner) for deflate on pypi.org.

my username there is: ThomasWaldmann

i would also offer to do next release in case you don't have time.

from deflate.

ThomasWaldmann avatar ThomasWaldmann commented on June 24, 2024

#29 will need a changelog entry after merge.

from deflate.

dcwatson avatar dcwatson commented on June 24, 2024

Biggest blocker (in my estimation) is probably Windows support. Need to make sure this tests/builds wheels on Windows in addition to macOS and Linux. It would also be good to have CI for #29 where we could make sure both paths work, but if that's prohibitive I'm fine doing it manually for now.

from deflate.

ThomasWaldmann avatar ThomasWaldmann commented on June 24, 2024

when i did some tags recently, it started the wheel build. so guess you could just merge and tag to trigger that.

i'll add a changelog entry now to the PR, so that todo is also done.

from deflate.

ThomasWaldmann avatar ThomasWaldmann commented on June 24, 2024

as long as pypi pkgconfig is NOT installed in your build action, it should fall back to the bundled code, so same behaviour as before.

from deflate.

ThomasWaldmann avatar ThomasWaldmann commented on June 24, 2024

found a nice cibuildwheel option to run tests... - trying that in the PR...

from deflate.

ThomasWaldmann avatar ThomasWaldmann commented on June 24, 2024

can we do a release ASAP? i have 1,2 PRs at borgbackup blocked by this...

from deflate.

dcwatson avatar dcwatson commented on June 24, 2024

Sorry for the radio silence, everyone in my house has been sick the past week.

I think this is pretty much ready to go. In thinking about it some more tonight, I think we should break out two packages, deflate and deflate-binary (like what psycopg2 did). Being able to configure how libdeflate is compiled or linked is great, but having pre-built binary wheels is also great for containers and places without compilers. So for 0.4 and forward, I think I will just skip building the wheels for the deflate package, call it the compiled version, and investigate how to make a deflate-binary package happen.

I know you seem to be in a hurry for a new release. Do you have any problem with a 0.4.0 release without binary wheels, until I can figure out a script for building a deflate-binary package from this repo?

from deflate.

ThomasWaldmann avatar ThomasWaldmann commented on June 24, 2024

Oh, i wish you all a quick recovery!

Timing is relaxed now, because I solved the issue in another way:

In borg, we already do a lot of interfacing to C libraries via Cython (e.g. to lz4, zstd, xxh64), so I decided to just do it in the same way for libdeflate. It's more consistent like that, was pretty easy (as it is only the one wrapper for crc32) and avoids adding a python dependency for deflate.

deflate python pkg was very useful to us for evaluating libdeflate performance on misc. platforms. On macOS, we'll use zlib.crc32 now, on Linux and all other platforms we'll use libdeflate_crc32.

from deflate.

ThomasWaldmann avatar ThomasWaldmann commented on June 24, 2024

About the binary package / wheels: I don't have strong opinions here.

from deflate.

william-silversmith avatar william-silversmith commented on June 24, 2024

Hi all! Would it be possible to get a new release of py310 and py311 wheels? Not super urgent, but would be good! Just need to bump the cibuildwheel version for GHA.

I know it's been a while, but I hope you're feeling better dcwatson!

from deflate.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.