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

Ensure data remains a shallow copy when added to plotter #2888

Merged
merged 4 commits into from
Jun 29, 2022

Conversation

akaszynski
Copy link
Member

Resolve #2461 by ensuring that a shallow copied dataset remains shallow when added to a plotter.

@codecov
Copy link

codecov bot commented Jun 28, 2022

Codecov Report

Merging #2888 (c6bc58e) into main (dcf18ff) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2888      +/-   ##
==========================================
- Coverage   94.11%   94.06%   -0.05%     
==========================================
  Files          76       76              
  Lines       16521    16521              
==========================================
- Hits        15549    15541       -8     
- Misses        972      980       +8     

banesullivan
banesullivan previously approved these changes Jun 28, 2022
tests/test_plotter.py Outdated Show resolved Hide resolved
Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com>
banesullivan
banesullivan previously approved these changes Jun 29, 2022
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.

Merging when CI comes back green

@banesullivan banesullivan enabled auto-merge (squash) June 29, 2022 15:34
@banesullivan banesullivan merged commit d5de70e into main Jun 29, 2022
@banesullivan banesullivan deleted the fix/ensure_remains_shallow_copy branch June 29, 2022 18:45
adeak added a commit to adeak/pyvista that referenced this pull request Jul 6, 2022
* upstream/main:
  Add setup and teardown functionality to plot_directive (pyvista#2907)
  Update hypothesis requirement from <6.48.4 to <6.49.2 (pyvista#2942)
  Bump lxml from 4.9.0 to 4.9.1 (pyvista#2936)
  Bump typing-extensions from 4.2.0 to 4.3.0 (pyvista#2939)
  Update hypothesis requirement from <6.48.3 to <6.48.4 (pyvista#2938)
  Bump trimesh from 3.12.6 to 3.12.7 (pyvista#2937)
  Update information of release (pyvista#2911)
  update pre-commit hooks (pyvista#2931)
  Handle invalid theme on init (pyvista#2917)
  Add `py.typed` Marker (pyvista#2904)
  [create-pull-request] update local intersphinx (pyvista#2926)
  Add tags with python commands to avoid misnumbering (pyvista#2416)
  Fix the url link of discretize (pyvista#2914)
  Add DataSet.cell_point_ids (pyvista#2897)
  Update hypothesis requirement from <6.48.2 to <6.48.3 (pyvista#2918)
  Ensure data remains a shallow copy when added to plotter (pyvista#2888)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Uh-oh! Something isn't working as expected.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Point data of a shallow-copied mesh is no longer a view after adding the mesh to a plotter
3 participants