-
Notifications
You must be signed in to change notification settings - Fork 25
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
build python 3.10 #101
build python 3.10 #101
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.
LGTM! A couple of minor points/questions about it, but these are more for my information!
@@ -24,33 +19,8 @@ def is_clang(bin): | |||
return not re.search(r'clang', output) is None | |||
|
|||
|
|||
def check_cython(): |
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.
Is it OK to delete this?
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.
Yes, because we now check build requirements with pyproject.toml
@@ -3,6 +3,6 @@ pytest | |||
matplotlib | |||
pytest | |||
pytest-cov | |||
vtk<9.1.0 | |||
vtk<9.1.0;python_version<"3.10" |
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.
So no VTK for Python 3.10?
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 isn’t anything official at the moment. There are pyvista wheels compiled for 3.10 and a discussion about it here:
pyvista/pyvista#2064
Build, but don't test, Python 3.10. Testing unavailable due to requirement for < 9.1.0 on xvfb.