Doc/add workflows #88
Conversation
…on for workflows and add fluent -> ensight -> adr example
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #88 +/- ##
=======================================
Coverage 75.94% 75.94%
=======================================
Files 16 16
Lines 5002 5002
Branches 1052 1052
=======================================
Hits 3799 3799
Misses 854 854
Partials 349 349 ☔ View full report in Codecov by Sentry. |
…uent. Add a few comments on live simulation
|
|
||
| import ansys.dynamicreporting.core as adr | ||
|
|
||
| os.environ["AWP_ROOT241"] = r"""C:\Program Files\ANSYS Inc\v241""" |
There was a problem hiding this comment.
I'm not a fan of these kinds of things being exposed in these scripts. Please let the script assume that a version of the Ansys installation can be found without this type of guidance.
| os.environ["AWP_ROOT241"] = r"""C:\Program Files\ANSYS Inc\v241""" | ||
|
|
||
| ansys_loc = r"""C:\Program Files\ANSYS Inc\v241""" | ||
| db_dir = r"""C:\fluent_workflow\db_dir""" |
There was a problem hiding this comment.
Can we use a Python tempdir for this or something similar? As written, it certainly has issues on Linux and probably most Windows machines :).
| # https://github.com/ansys/pydynamicreporting/tree/main/doc/source/_data. | ||
| # | ||
|
|
||
| ctx_file = r"""C:\fluent_workflow\context1a.ctx""" |
There was a problem hiding this comment.
Hardcoded pathname that would be easy to miss if you were trying to run this.
| ensession.ensight.part.select_all() | ||
| ensession.ensight.savegeom.format("avz") | ||
| ensession.ensight.savegeom.select_all_steps() | ||
| ensession.ensight.savegeom.save_geometric_entities(r"""C:\fluent_workflow\export_avz""") |
There was a problem hiding this comment.
I guess in general, using a temporary directory would be better.
Add cheat sheet at the home page of the docs
Add a section for the workflow examples, and add the Fluent -> Ensight -> ADR example.