Skip to content

Conversation

@jsalant22
Copy link
Collaborator

@jsalant22 jsalant22 commented Sep 5, 2025

Fix the check for a read-only directory

Description

Example would give a warning about a read-only directory when trying to Generate Results

Issue linked

(https://tfs.ansys.com:8443/tfs/ANSYS_Development/Portfolio/_boards/board/t/EBU-EMIT/Stories/?workitem=1263233)

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.

Fix the check for a read-only directory
Copy link
Collaborator

@anspraksaph anspraksaph left a comment

Choose a reason for hiding this comment

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

For this PR, I tested with PyAedt main branch (Commit hash: cc29743c5b51c81b2e7ed84c211ed955405f3e12) and Aedt 26.1, this defect is fixed.

  • interference_type.py
image

@Samuelopez-ansys
Copy link
Member

Hi @jsalant22 @anspraksaph

I think these EMIT examples does not fit the example structure. If I understand well they are building an UI .

Then, for me they are extension. Which are in PyAEDT.

correct me if I am wrong, in addition to, you are using pyside6, which is a dependency that it is not part of PyAEDTA.

Extensions use tkinter, which is part of the python installation.

please, could you consider migrate these Workflows to extensions? Ping me if you need more info.

examples should be end to end workflows, that you can run as python scripts or Jupyter notebook scripts

CC @SMoraisAnsys

@jsalant22
Copy link
Collaborator Author

@Samuelopez-ansys Can you send me a link to the extensions or a template? Are they part of the main PyAedt repo?

@Samuelopez-ansys Samuelopez-ansys marked this pull request as draft September 8, 2025 13:48
auto-merge was automatically disabled September 8, 2025 13:48

Pull request was converted to draft

@jsalant22
Copy link
Collaborator Author

@jsalant22 Thanks! This example and UI is ~2 years old, that's why we didn't implement as an extension.

We'll look into converting it though!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants