fix: avoid changing entities ids after plotting

[germa89]

Description

Avoid plotting changing the id of the returned elements.

Issue linked

Close #3372
Close #3374

Checklist

• I have visited and read Develop PyMAPDL.
• I have tested my changes locally
• I have added necessary documentation or updated existing documentation.
• I have followed the PyMAPDL coding style guidelines.
• I have added appropriate unit tests that also ensure the minimal coverage criteria.
• I have reviewed my changes before submitting this pull request.
• I have linked the issue or issues that are solved to the PR if any.
• I have assigned this PR to myself.
• I have marked this PR as draft if it is not ready to be reviewed yet.
• I have made sure that the title of my PR follows Commit naming conventions (e.g. feat: adding new MAPDL command)

[ansys-reviewer-bot]

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

[codecov]

Codecov Report

Attention: Patch coverage is 86.04651% with 6 lines in your changes missing coverage. Please review.

Project coverage is 86.85%. Comparing base (1a1cfd4) to head (66de578).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3421      +/-   ##
==========================================
+ Coverage   85.08%   86.85%   +1.76%     
==========================================
  Files          55       55              
  Lines        9977     9972       -5     
==========================================
+ Hits         8489     8661     +172     
+ Misses       1488     1311     -177     

[germa89]

For some reason I keep getting ansys.mapdl.core.errors.MapdlConnectionError: The channel is not alive. on some tests:

@pytest.mark.parametrize(
    "plot_func,entity,gen_func,arg1,arg2",
    [
        ("vplot", "VOLU", "block", (0, 1, 0, 1, 0, 1), (1, 2, 1, 2, 1, 2)),
        # Uncommenting the following two lines, raise an exception for channel not
        # alive. See #3421
        # ("aplot", "AREA", "rectng", (0, 1, 0, 1), (1, 2, 1, 2)),
        # ("lplot", "LINE", "l", (1, 1, 1), (1, -1, 1)),
        ("kplot", "KP", "k", ("", 0, 0, 0), ("", 1, 1, 1)),
    ],
)
def test_xplot_not_changing_geo_selection_components(
....

I haven't been able to found a solution, but it seems other tests pass. So I'm going to keep them comment them, and hopefully I can come back in the future.

[github-actions]

Hello! 👋

Your PR is changing the image cache. So I am attaching the new image cache in a new commit.

This commit does not re-run the CICD workflows (since no changes are made in the codebase) therefore you will see the actions showing in their status Expected — Waiting for status to be reported. Do not worry. You commit workflow is still running here 😄

You might want to rerun the test to make sure that everything is passing. You can retrigger the CICD sending an empty commit git commit -m "chore: empty comment to trigger CICD" --allow-empty.

You will see this message everytime your commit changes the image cache but you are not attaching the updated cache. 🤓

[germa89]

@pyansys-ci-bot LGTM.

[pyansys-ci-bot]

✅ Approving this PR because germa89 said so in here 😬