Skip to content

Conversation

@PProfizi
Copy link
Contributor

@PProfizi PProfizi commented Sep 25, 2025

Modifies .ci/generate_operators_doc.py by:

  • adding an --output_path argument to select where to write the operator specs doc.
  • fix the location of the generated toc.yml relative to the doc root.
  • adds the operator_doc dependency group to list jinja2 as optional requirement (requires a release for it to work)

Also allow to generate the operator doc sources without cloning pydpf-core:

  • moves the script to a module of the library (ansys.dpf.core.documentation.py)
  • adds an entry-point to the library to install a command in the terminal (dpf_generate_operators_doc) upon installation of the library
  • packages the templates doc/source/operators_doc/operator-specifications/operator_doc_template.md and doc/source/operators_doc/operator-specifications/toc_template.j2

@PProfizi PProfizi requested a review from AnsMelanie September 25, 2025 14:10
@PProfizi PProfizi self-assigned this Sep 25, 2025
@codecov
Copy link

codecov bot commented Sep 25, 2025

Codecov Report

❌ Patch coverage is 94.69027% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.25%. Comparing base (7d910e4) to head (7202d26).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2635      +/-   ##
==========================================
+ Coverage   84.14%   84.25%   +0.11%     
==========================================
  Files          91       92       +1     
  Lines       10709    10819     +110     
==========================================
+ Hits         9011     9116     +105     
- Misses       1698     1703       +5     

@PProfizi PProfizi added the CI/CD Related to CI/CD label Sep 26, 2025
@PProfizi PProfizi changed the title ci(doc): improve operator documentation generation script feat(doc): improve operator Markdown documentation generation Sep 26, 2025
@PProfizi PProfizi added enhancement New feature or request and removed CI/CD Related to CI/CD labels Sep 26, 2025
@github-actions github-actions bot added the CI/CD Related to CI/CD label Sep 26, 2025
@PProfizi PProfizi force-pushed the ci/automate_devreldocs_update branch from 439fedf to de8b9be Compare September 26, 2025 13:07
/ "dpf"
/ "plugins"
/ "dpf_composites"
/ "composite_operators.dll",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We should make it compatible with Linux. Is this a priority though?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is there any reason to restrict it to Windows?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

-> Sound plugin exists only for Windows, but that is all

@PProfizi
Copy link
Contributor Author

Check if it works for a remote server.

@github-actions
Copy link
Contributor

Some tests with 'continue-on-error: true' have failed:

  • PyDPF-Post API tests on ubuntu-latest

Created by continue-on-error-comment

@PProfizi PProfizi merged commit 1e18868 into main Sep 30, 2025
46 checks passed
@PProfizi PProfizi deleted the ci/automate_devreldocs_update branch September 30, 2025 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Related to CI/CD enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants