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

Implement fixes for VTK 9.2.0rc #2863

Merged
merged 9 commits into from
Jun 28, 2022
Merged

Implement fixes for VTK 9.2.0rc #2863

merged 9 commits into from
Jun 28, 2022

Conversation

akaszynski
Copy link
Member

@akaszynski akaszynski commented Jun 26, 2022

Adds fixes for vtk 9.2 release candidate.

Regressions (VTK side):

Other changes:

  • Split up chart tests to aid in debugging.
  • Change GetError to GetThresholdError for image comparison.
  • Add note regarding flipping of cubemap direction between VTK 9.1.0 and 9.2.0.
  • Also adds Python 3.10 to testing workflow.

@github-actions github-actions bot added dependencies Pull requests that update a dependency file maintenance Low-impact maintenance activity enhancement Changes that enhance the library labels Jun 26, 2022
@akaszynski akaszynski changed the title add fixes for py310 Implement fixes for VTK 9.2.0rc Jun 26, 2022
@codecov
Copy link

codecov bot commented Jun 26, 2022

Codecov Report

Merging #2863 (c137fb8) into main (22dc22e) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2863      +/-   ##
==========================================
- Coverage   94.09%   94.08%   -0.01%     
==========================================
  Files          76       76              
  Lines       16426    16443      +17     
==========================================
+ Hits        15456    15471      +15     
- Misses        970      972       +2     

tkoyama010
tkoyama010 previously approved these changes Jun 26, 2022
Copy link
Member

@tkoyama010 tkoyama010 left a comment

Choose a reason for hiding this comment

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

LGTM

.github/workflows/testing-and-deployment.yml Show resolved Hide resolved
pyvista/core/filters/data_set.py Outdated Show resolved Hide resolved
pyvista/utilities/helpers.py Show resolved Hide resolved
pyvista/utilities/regression.py Outdated Show resolved Hide resolved
requirements_test.txt Show resolved Hide resolved
tests/test_regression.py Outdated Show resolved Hide resolved
tests/utilities/test_reader.py Outdated Show resolved Hide resolved
Co-authored-by: Andras Deak <adeak@users.noreply.github.com>
@akaszynski
Copy link
Member Author

@banesullivan, could you review so we could have this merged tomorrow? I'm hoping for a v0.35.1 (not v0.35.0 because I screwed that release up) by the end of the week.

Copy link
Member

@banesullivan banesullivan left a comment

Choose a reason for hiding this comment

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

LGTM

@akaszynski akaszynski merged commit d8806f7 into main Jun 28, 2022
@akaszynski akaszynski deleted the feat/py310 branch June 28, 2022 23:22
akaszynski added a commit that referenced this pull request Jun 29, 2022
* add fixes for py310

* add windows and mac to testing

* improve coverage

* Update pyvista/core/filters/data_set.py

Co-authored-by: Andras Deak <adeak@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: Andras Deak <adeak@users.noreply.github.com>

* add crushed can example

Co-authored-by: Andras Deak <adeak@users.noreply.github.com>
akaszynski added a commit that referenced this pull request Jun 30, 2022
* Fix imports of vtkExtractEdges and vtkCellTreeLocator (#2585)

These classes have been moved and imports failed. This checks for VTK versions/builds and imports them accordingly.

* Implement fixes for VTK 9.2.0rc (#2863)

* add fixes for py310

* add windows and mac to testing

* improve coverage

* Update pyvista/core/filters/data_set.py

Co-authored-by: Andras Deak <adeak@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: Andras Deak <adeak@users.noreply.github.com>

* add crushed can example

Co-authored-by: Andras Deak <adeak@users.noreply.github.com>

* bump to v0.34.2

* don't use cache

* Fix use of a slice tuple for numpy 1.23 (#2726)

* remove more cache

Co-authored-by: Håkon Strandenes <h.strandenes@km-turbulenz.no>
Co-authored-by: Andras Deak <adeak@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement Changes that enhance the library maintenance Low-impact maintenance activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants