Skip to content

FIX: Release Desktop for lumped element example#555

Open
ramin4667 wants to merge 9 commits into
mainfrom
fix/release_desktop_for_lumped_element_example
Open

FIX: Release Desktop for lumped element example#555
ramin4667 wants to merge 9 commits into
mainfrom
fix/release_desktop_for_lumped_element_example

Conversation

@ramin4667

Copy link
Copy Markdown
Collaborator

Description

This PR updates the lumped element filter example by adding comments and the release desktop command to close the opened desktops.

Issue linked

Left the desktop open after the end of the example.

Checklist

Please complete the following checklist before submitting your pull request:

  • I have followed the example template and guide lines to add/update an example.
  • I have tested the example locally and verified that it is working with the latest version of AEDT.
  • I have verified that these changes to the best of my knowledge do not introduce any security vulnerabilities.

Comment thread examples/high_frequency/radiofrequency_mmwave/lumped_element.py Outdated
Comment thread examples/high_frequency/radiofrequency_mmwave/lumped_element.py
SMoraisAnsys
SMoraisAnsys previously approved these changes Apr 1, 2026

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

LGTM, just a minor comment as . shouldn't be in a section title.

Comment thread examples/high_frequency/radiofrequency_mmwave/lumped_element.py Outdated
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Apr 8, 2026
Comment thread examples/high_frequency/radiofrequency_mmwave/lumped_element.py Outdated
Comment thread .pre-commit-config.yaml
Comment on lines -39 to -45

- repo: https://github.com/ansys/pre-commit-hooks
rev: v0.5.2
hooks:
- id: add-license-headers
args:
- --start_year=2024

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.

Can you revert back this change ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants