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

Fixing doc build #2570

Closed
wants to merge 2 commits into from
Closed

Fixing doc build #2570

wants to merge 2 commits into from

Conversation

germa89
Copy link
Collaborator

@germa89 germa89 commented Dec 15, 2023

I have seen this error in doc builds:

../../examples/00-mapdl-examples/exhaust_manifold_thermal_stress.py unexpectedly failed to execute correctly:

    Traceback (most recent call last):
      File "/home/runner/work/pymapdl/pymapdl/examples/00-mapdl-examples/exhaust_manifold_thermal_stress.py", line 164, in <module>
        mapdl.post_processing.plot_nodal_eqv_stress()  # Plot equivalent stress
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/post.py", line 2072, in plot_nodal_eqv_stress
        return self._plot_point_scalars(
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/post.py", line 606, in _plot_point_scalars
        surf = self._mapdl.mesh._surf
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/mesh_grpc.py", line 76, in _surf
        self._surf_cache = self._grid.extract_surface()
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/misc.py", line 1116, in wrapper
        return function(self, *args, **kwargs)
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/mesh_grpc.py", line 716, in _grid
        self._grid_cache = self._parse_vtk(force_linear=True)
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/mesh_grpc.py", line 783, in _parse_vtk
        return _parse_vtk(
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/mesh/mesh.py", line 172, in _parse_vtk
        for key, item in mesh.element_components.items():
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/mesh_grpc.py", line 326, in element_components
        return self._mapdl.components._get_all_components_type("ELEM")
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/component.py", line 511, in _get_all_components_type
        item = self.__getitem__(each_comp)
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/component.py", line 311, in __getitem__
        output = self._mapdl._parse_cmlist_indiv(key, cmtype)
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/mapdl.py", line 5021, in _parse_cmlist_indiv
        items = [int(each) for each in items]
      File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/ansys/mapdl/core/mapdl.py", line 5021, in <listcomp>
        items = [int(each) for each in items]
    ValueError: invalid literal for int() with base 10: '*****MAPDL'

This PR fix that issue.

@ansys-reviewer-bot
Copy link
Contributor

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

@ansys-reviewer-bot review

@github-actions github-actions bot added the BUG label Dec 15, 2023
@germa89
Copy link
Collaborator Author

germa89 commented Dec 15, 2023

LGTM

Copy link
Contributor

@pyansys-ci-bot pyansys-ci-bot left a comment

Choose a reason for hiding this comment

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

✅ Approving this PR because germa89 said so in here 😬

@germa89 germa89 self-assigned this Dec 15, 2023
Copy link

codecov bot commented Dec 15, 2023

Codecov Report

Merging #2570 (f3ad8e4) into main (00d49f6) will decrease coverage by 0.05%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

❗ Current head f3ad8e4 differs from pull request most recent head 4db0bac. Consider uploading reports for the commit 4db0bac to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2570      +/-   ##
==========================================
- Coverage   82.52%   82.48%   -0.05%     
==========================================
  Files          44       44              
  Lines        9054     9048       -6     
==========================================
- Hits         7472     7463       -9     
- Misses       1582     1585       +3     

@germa89
Copy link
Collaborator Author

germa89 commented Dec 15, 2023

Closing because merged in #2558

@germa89 germa89 closed this Dec 15, 2023
@germa89 germa89 deleted the fix/doc-builds branch December 15, 2023 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants