ansys / pyensight

Python API for EnSight
http://ensight.docs.pyansys.com
MIT License
6 stars 2 forks source link

Improved error handling #442

Closed randallfrank closed 1 month ago

randallfrank commented 1 month ago

Avoid EnSight / OV hang when running out of disk space. Fixed issues with calculator function docs. Added GLB file generation stub in testing fixtures.

mariostieriansys commented 1 month ago

@randallfrank minor comment Should we move the GLButils class outside of confest.py and just import it? SO the module would still be under tests, it is just to make the code cleaner

codecov-commenter commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.21%. Comparing base (4cece7a) to head (8a43585). Report is 4 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #442 +/- ## ========================================== - Coverage 85.23% 85.21% -0.03% ========================================== Files 19 19 Lines 3488 3489 +1 Branches 443 473 +30 ========================================== Hits 2973 2973 - Misses 430 433 +3 + Partials 85 83 -2 ``` | [Flag](https://app.codecov.io/gh/ansys/pyensight/pull/442/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ansys) | Coverage Δ | | |---|---|---| | [](https://app.codecov.io/gh/ansys/pyensight/pull/442/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ansys) | `85.21% <ø> (-0.03%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ansys#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

randallfrank commented 1 month ago

@randallfrank minor comment Should we move the GLButils class outside of confest.py and just import it? SO the module would still be under tests, it is just to make the code cleaner

Done!