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

Minimum Required Versions - Python >=3.7; Numpy >= 1.20 #235

Closed
wetzelj opened this issue Oct 18, 2022 · 10 comments
Closed

Minimum Required Versions - Python >=3.7; Numpy >= 1.20 #235

wetzelj opened this issue Oct 18, 2022 · 10 comments

Comments

@wetzelj
Copy link
Contributor

wetzelj commented Oct 18, 2022

Hey @vsoch... I've been stale for a bit and have let my application lag at 0.2.28. Looking to update now and came across this:

With the implementation of version 0.2.34 and the use of numpy.typing, a minimum python and numpy dependency has been introduced. Typing was introduced in numpy version 1.20; numpy v1.20 requires Python >= 3.7. Since you're currently working on #234, I thought you may want to note these dependencies as well.

And yes, my head is hanging in shame at still being on Python 3.6... but that should be fixed shortly. :)

@vsoch
Copy link
Member

vsoch commented Oct 18, 2022

haha, no feel for shame! We have some projects that are also stuck on 3.6 (I believe) because of RHEL systems that need to be supported. This is probably a silly question - but where/how is the best way to define a Python version? Is the setting in the setup.py metadata actually honored?

@wetzelj
Copy link
Contributor Author

wetzelj commented Oct 18, 2022

I didn't know the answer to that either, but it looks like python_requires can be utilized to force a minimum python version.

@vsoch
Copy link
Member

vsoch commented Oct 18, 2022

Thanks @wetzelj ! I'll add these tweaks tonight, and bump the version more too.

@vsoch
Copy link
Member

vsoch commented Oct 19, 2022

okay should be pushed!

@wetzelj
Copy link
Contributor Author

wetzelj commented Oct 19, 2022

What version of Python do you typically target for deid?

@vsoch
Copy link
Member

vsoch commented Oct 19, 2022

I would say likely 3.7? On my own machine I always have a newer one (I use anaconda and venv and nuke the entire thing once in a while) so I am usually at 3.8x or 3.9x.

@vsoch
Copy link
Member

vsoch commented Oct 21, 2022

@vsoch
Copy link
Member

vsoch commented Nov 22, 2022

@wetzelj good to close here?

@wetzelj
Copy link
Contributor Author

wetzelj commented Nov 22, 2022

Yep. Thank you!

@wetzelj wetzelj closed this as completed Nov 22, 2022
@vsoch
Copy link
Member

vsoch commented Nov 22, 2022

It was all you - so thank you!

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

No branches or pull requests

2 participants