Skip to content
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

CI: Temporary removal of PyAEDT dotnet tests #987

Closed

Conversation

SMoraisAnsys
Copy link
Collaborator

Following #986, this PR aims at avoiding the tests that are currently failing when releasing.

For visibility and future reference, here is a mini reproducer of the failure within the VM

import os
from ansys.aedt.core.generic.grpc_plugin_dll_class import AEDT
os.environ["DesktopPluginPyAEDT"] = "/opt/AnsysEM/v242/Linux64/PythonFiles/DesktopPlugin"
AEDT(os.environ["DesktopPluginPyAEDT"])

from pythonnet import load
load(
    "coreclr",
    runtime_config=str("/usr/share/dotnet/sdk/6.0.428/dotnet.runtimeconfig.json"),
    dotnet_root=str("/usr/share/dotnet")
)

Note that removing the first 4 lines of the previous snippet allows to load coreclr correctly.

@github-actions github-actions bot added maintenance Package and maintenance related ci/cd labels Jan 17, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jan 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.67%. Comparing base (e804b3a) to head (8d9eda6).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #987   +/-   ##
=======================================
  Coverage   82.67%   82.67%           
=======================================
  Files         160      160           
  Lines       21130    21130           
=======================================
  Hits        17470    17470           
  Misses       3660     3660           

@SMoraisAnsys
Copy link
Collaborator Author

Closing as it shouldn't be required anymore once ansys/pyaedt#5673 is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/cd maintenance Package and maintenance related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants