-
Notifications
You must be signed in to change notification settings - Fork 142
feat: (PR 4122) adding OSRESULT command to the misc loads #4131
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4131 +/- ##
==========================================
- Coverage 91.35% 91.31% -0.04%
==========================================
Files 189 189
Lines 15650 15673 +23
==========================================
+ Hits 14297 14312 +15
- Misses 1353 1361 +8 🚀 New features to boost your workflow:
|
|
Possible bug on But I get: Details
Output |
Hi @germa89, Thank you for adding the unit test and accepting the PR request. I think your input file has a typo here. The last line should be "PRESOL,SRES,SY" instead of "PRESOL,SRES,Y". This is the example from the documentation: ! When issuing postprocessing commands to retrieve selected |
|
@mbbatukan you are right, the typo in the APDL is correct. However, even after correcting it I still get: And the unit tests do not have that typo: @pytest.mark.xfail(
reason="Bug? PRESOL cannot retrieve osresult data",
strict=True,
)
def test_osresult_presol(mapdl, solved_box):
mapdl.solution()
mapdl.outres("ALL", "NONE")
mapdl.osresult("S", "Y", "ALL") # Stress Y component
mapdl.solve()
mapdl.post1()
assert mapdl.presol("SRES", "SY") |
|
Yes, I also encountered the same error after fixing the APDL script. |
This PR is a mirror pull request created from feat: adding OSRESULT command to the misc loads to allow the code to access PyMAPDL CICD secrets.
Check the original PR made by @mbbatukan for more details.
Closes #4122
Original pull request
feat: adding OSRESULT command to the misc loads
Description
The OSRESULT command has been added to the misc loads module to support additional result data.
Please refer to: https://ansyshelp.ansys.com/public/account/secured?returnurl=/Views/Secured/corp/v242/en/ans_cmd/Hlp_C_OSRESULT.html
Summary by Sourcery
New Features: