-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Update Numpy minimum version to 1.17.0 #3156
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No objections from me.
NEP 29 implies the Numpy project has already (as of January 2021) dropped support for every version earlier It'd be logical to never do a formal release that specifies an expired-support version of Numpy - so I'd be fine bumping to 1.17 ASAP. We may even want to go further, and always ensure our releases specify a Numpy version that has at least N months of support remaining, where N could be 3, 6, or even 12. |
Okay, merged. Congrats on your first Gensim contribution @PrimozGodec ⭐ @gojomo is there a way to automate that? If the process relies on us (or users) manually submitting version edits to |
I don't know of a standard way to integrate a project's "oldest supported" or "oldest supported as of a certain date" version - it seems those expectations are still somewhat informally specified, as through that 'NEP', rather than rigorously specified. I could imagine some new convention where a project, at a well-known & relatively-secure URL under its management, might maintain text fragments specifying key transitions of support. For example, perhaps Barring something that reliable, adding to a checklist for new releases a quick review of major prerequisites (like |
Thanks, makes sense. Could you add that step to the release checklist please (with a link to this PR, for context)? |
Where is a release checklist, if any? (A search of the repo reveals no files ith 'checklist' in them.) |
I've added an item with the gist-of-the-check at: https://github.com/RaRe-Technologies/gensim/wiki/Developer-page#prepare - review & adjust as desired. Since our earlier discussion, I came across scipy's It seems more focused on being a build-prereq for wheels, not an end-user-install-time prereq - so I'm unsure whether it changes our concerns here.... but it's interesting to know about, as a formalized way to express upstream-dependency lifecycle states. |
Yeah, we use oldest-supported-numpy for building gensim wheels for releases, but as you've stated, they solve a slightly different problem than the one mentioned by the OP. |
Based on the discussion in #3137 I am updating the minimum NumPy version to 1.17.0. It is required since otherwise users who install Gensim in the environment with an older NumPy version get
AttributeError: module 'numpy.random' has no attribute 'default_rng'
error.