MatthewFlamm

Results 336 comments of MatthewFlamm

This is already available when I try it. For example: ```python streams = mesh.streamlines() print(streams.array_names) ``` gives: ```txt ['velocity', 'IntegrationTime', 'Vorticity', 'Rotation', 'AngularVelocity', 'Normals', 'ReasonForTermination', 'SeedIds'] ``` All data arrays...

Does the resample filter do what you want? https://docs.pyvista.org/examples/01-filter/resample.html Edit: this is the _sample_ filter, the title of this example had confused me.

You can try: ``` mesh["my_vector_name"] = np.column_stack((mesh.get_array('Mx'), mesh.get_array('My'), mesh.get_array('Mz')) mesh.set_active_vectors("my_vector_name") # not strictly neccessary, but it will replicate the mesh.vectors usage. ``` See here: https://github.com/pyvista/pyvista/blob/274261f684aad2d9f2ce5dcf9c864a7cf2ca2c5a/pyvista/core/dataset.py#L247-L258

Your method is basically the same as mine above, when doing `mesh["M_vec"] = vector_data`, but your usage will only work for point_data, whereas this usage will work for both point...

I was incorrect, `mesh.vectors = cell_data` will not work since it adds to `mesh.point_arrays`. This is a bug, although I'd prefer this whole functionality to be removed.

How are you saving and reading the files? This works for me: ```python import pyvista as pv import numpy as np sphere = pv.Sphere() cube = pv.Cube() data = np.array([1.2,...

I find https://peps.python.org/pep-0565/ to be a bit confusing in recommendations. This section at first seems pretty cut and dry in which to use: > For library and framework authors that...

I have run into cases where `SetInputArrayToProcess` leads to a bug when multiple filters of the same type are applied sequentially. See #1463 and #1475. Using `active_scalars` fixed that bug....

I'm +1 on delegating logic to another package here. I haven't gone through in detail, but it looks like the ability to use a local git repo will be removed...

> > I haven't gone through in detail, but it looks like the ability to use a local git repo will be removed with this PR. > > @MatthewFlamm could...