Skip to content

FIX: test_export_to_aedt_test_FAIL_New#7861

Open
ramin4667 wants to merge 21 commits into
mainfrom
D1457509__PyAEDT_test_export_to_aedt_test_FAIL_New
Open

FIX: test_export_to_aedt_test_FAIL_New#7861
ramin4667 wants to merge 21 commits into
mainfrom
D1457509__PyAEDT_test_export_to_aedt_test_FAIL_New

Conversation

@ramin4667

@ramin4667 ramin4667 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Description

New folder for test.

Issue linked

COM connection issue

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(s) that are solved by the PR if any.
  • I have assigned this PR to myself.
  • I have added the minimum version decorator to any new backend method implemented.
  • I have agreed with the Contributor License Agreement (CLA).

@ramin4667
ramin4667 requested a review from a team as a code owner June 30, 2026 18:23
@github-actions github-actions Bot added the maintenance Package and maintenance related label Jun 30, 2026
@Samuelopez-ansys

Copy link
Copy Markdown
Member

Hi @ramin4667 , I think you should plan to move to gRPC in the short term. dotnet is going to be removed from PyAEDT this year because PyEDB does not need it anymore. And pythonnet is not maintained anymore, and PyAnsys team had to fork it to keep it alive.

@maxcapodi78 @SMoraisAnsys What do you think?

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

Here are some minor comments, thanks for the contribution !

Comment thread .github/workflows/manual_draft.yml Outdated
Comment thread doc/changelog.d/7860.fixed.md Outdated
Comment thread src/ansys/aedt/core/desktop.py
Comment thread pyproject.toml Outdated
Comment thread pyproject.toml Outdated
Comment thread uv.lock Outdated
@ramin4667
ramin4667 force-pushed the D1457509__PyAEDT_test_export_to_aedt_test_FAIL_New branch from a816e7c to 50826c5 Compare July 3, 2026 16:39
…IL_New

# Conflicts:
#	.github/workflows/ci-release.yml
#	.github/workflows/ci_cd.yml
#	.github/workflows/nightly-tests.yml
#	.github/workflows/sync-release.yml
#	.github/workflows/update-testmondata-cache.yml
#	.github/workflows/weekly-tests.yml
@Samuelopez-ansys

Copy link
Copy Markdown
Member

Hi @ramin4667 , tests are failing locally, I do not understand how they will work if use_grpc is set to True.

@Samuelopez-ansys

Copy link
Copy Markdown
Member

How can I run these tests locally?

@ramin4667

Copy link
Copy Markdown
Contributor Author

Hi @ramin4667 , tests are failing locally, I do not understand how they will work if use_grpc is set to True.

@Samuelopez-ansys , you are right, I set it to False.

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

Labels

maintenance Package and maintenance related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants