feat: enhance facet extraction functions to return element_ids and pa…

[rokamoto26]

…rt_ids

fix: change get_polydata and extract_solids to display solid elements correctly
fix: update test_plotter.py according to the update of deck_plotter.py

[koubaa]

Thanks for the contribution, the deck plotter is indeed under-developed. Could you please fix the failing test case?

[rokamoto26]

HI @koubaa, I fixed the test case and all checks have passed. Could you review the updated code?

[koubaa]

@rokamoto26 I downloaded the html documentation from the documentation build check, and some of the images look different. For example in your branch the beer can example plot looks like this:

See the current doc image here:
https://dyna.docs.pyansys.com/version/stable/examples/Buckling_Beer_Can/plot_beercan.html#view-the-model

[rokamoto26]

Hi @koubaa, thank you for reviewing!

This is just the part ID being plotted. Since the part ID is stored as a scalar value, it is plotted according to the PyVista default behavior.
https://docs.pyvista.org/api/plotting/_autosummary/pyvista.plotter.add_mesh#pyvista.Plotter.add_mesh

It is possible to control this behavior in deck_plotter, but if it differs from the PyVista specification, it may cause confusion for users.

[koubaa]

In that case, the display of the part ID should be opt-in, not default.

[rokamoto26]

@koubaa Thank you for your comment. I changed plot behavior. Could you review again?
Note that the images of the Taylor Bar Impact and John Reid pipe examples have changed, as the previous ones were incorrect.

[koubaa]

I verified the new visualizations, and they are a clear improvement. Thank you for your contribution!