Skip to content

FEAT: SAM Bot #6145

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

Open
wants to merge 22 commits into
base: main
Choose a base branch
from
Open

FEAT: SAM Bot #6145

wants to merge 22 commits into from

Conversation

Samuelopez-ansys
Copy link
Member

Description

Add SAM Bot (Smart AEDT Manager) for managing and launching AEDT extensions.

This commit introduces a lightweight, floating GUI tool called SAM Bot, designed to provide a graphical interface for launching PyAEDT extension scripts. The bot displays a draggable icon (loaded from assets) and uses a context menu (configured via a TOML file) to launch categorized scripts using a specified or default Python interpreter.

Key features:

  • Loads configuration from environment variable or default TOML file
  • Automatically detects appropriate Python interpreter
  • Dynamically builds right-click context menu for script launch
  • Provides a help window describing usage and features
  • Designed to work both as a PyInstaller bundle and as a standalone script

This utility enhances usability for non-CLI users and facilitates quick access to automation tools in the Ansys Electronics Desktop ecosystem. 🤖

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).

@ansys-reviewer-bot
Copy link
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@github-actions github-actions bot added maintenance Package and maintenance related enhancement New features or code improvements labels May 12, 2025
@Samuelopez-ansys
Copy link
Member Author

@SMoraisAnsys This is using subprocess and bandit is complaining, please let me know what you think :)

Copy link

codecov bot commented May 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.24%. Comparing base (7e2637d) to head (95f24e3).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6145   +/-   ##
=======================================
  Coverage   85.24%   85.24%           
=======================================
  Files         170      170           
  Lines       64119    64119           
=======================================
  Hits        54656    54656           
  Misses       9463     9463           
🚀 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.

Copy link
Collaborator

@SMoraisAnsys SMoraisAnsys left a comment

Choose a reason for hiding this comment

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

I'll add request changes to ensure that we are aligned on that PR. There is a need for some discussion before merging it or not (independently from the code changes).

Great feature though :)

@Samuelopez-ansys Samuelopez-ansys requested a review from a team as a code owner May 20, 2025 08:03
# Conflicts:
#	.github/workflows/ci_cd.yml
@Samuelopez-ansys
Copy link
Member Author

Hi @SMoraisAnsys and @maxcapodi78 , please could you take a look to this PR?

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 maintenance Package and maintenance related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants