Skip to content

FEAT: Add panels command to manage PyAEDT panels in AEDT + tests - #6886

Merged
Samuelopez-ansys merged 29 commits into
mainfrom
feat/cli-add-panels
Nov 24, 2025
Merged

Samuelopez-ansys merged 29 commits into
mainfrom
feat/cli-add-panels

Conversation

@eblanco-ansys

Copy link
Copy Markdown
Contributor

Description

Add panels command to manage PyAEDT panels in AEDT + tests

Checklist

  • I have tested my changes locally.
  • I have added necessary documentation or updated existing documentation.
  • I have followed the coding style guidelines of this project.
  • I have added appropriate tests (unit, integration, system).
  • I have reviewed my changes before submitting this pull request.
  • I have linked the issue or issues that are solved by the PR if any.
  • I have agreed with the Contributor License Agreement (CLA).

@eblanco-ansys
eblanco-ansys requested a review from a team as a code owner November 17, 2025 15:25
@github-actions github-actions Bot added the enhancement New features or code improvements label Nov 17, 2025
Comment thread src/ansys/aedt/core/cli.py Outdated
Comment thread src/ansys/aedt/core/cli.py Outdated
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Comment thread src/ansys/aedt/core/cli.py Outdated
eblanco-ansys and others added 2 commits November 17, 2025 16:32
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Comment thread src/ansys/aedt/core/cli.py Outdated
Comment thread src/ansys/aedt/core/cli.py Outdated
eblanco-ansys and others added 2 commits November 17, 2025 16:36
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
@eblanco-ansys

Copy link
Copy Markdown
Contributor Author

@PipKat Thanks a lot for the suggestions! Appreciated

Comment thread src/ansys/aedt/core/cli.py Outdated
Comment thread src/ansys/aedt/core/cli.py Outdated
Comment thread src/ansys/aedt/core/cli.py Outdated
Comment thread src/ansys/aedt/core/cli.py Outdated
Comment thread src/ansys/aedt/core/cli.py Outdated
eblanco-ansys and others added 9 commits November 17, 2025 16:40
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
@codecov

codecov Bot commented Nov 18, 2025 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.80488% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.09%. Comparing base (f1704ad) to head (3a4b732).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6886      +/-   ##
==========================================
+ Coverage   80.21%   81.09%   +0.87%     
==========================================
  Files         247      247              
  Lines       78104    78177      +73     
==========================================
+ Hits        62655    63399     +744     
+ Misses      15449    14778     -671     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@SMoraisAnsys SMoraisAnsys left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm wondering if adding this to the CLI is needed or if we shouldn't. Do we have users / developers that would leverage that feature ? I've left some minor comments

Comment thread tests/unit/test_cli.py Outdated
Comment thread src/ansys/aedt/core/cli.py
@eblanco-ansys

Copy link
Copy Markdown
Contributor Author

I'm wondering if adding this to the CLI is needed or if we shouldn't. Do we have users / developers that would leverage that feature ? I've left some minor comments

The idea behind this feature is to facilitate how users add the panels when doing manually. Currently, they need to type this inside python console and replace with the correct values:
from ansys.aedt.core.extensions.installer.pyaedt_installer import add_pyaedt_to_aedt add_pyaedt_to_aedt(“your_aedt_version", r“path_to_personalib")
With this new command they will be guided and get more feedback

@Samuelopez-ansys
Samuelopez-ansys merged commit dbdfdcb into main Nov 24, 2025
40 checks passed
@Samuelopez-ansys
Samuelopez-ansys deleted the feat/cli-add-panels branch November 24, 2025 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New features or code improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants