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

Migration common toolkit #193

Merged
merged 34 commits into from
May 13, 2024
Merged

Migration common toolkit #193

merged 34 commits into from
May 13, 2024

Conversation

gmalinve
Copy link
Collaborator

No description provided.

# Conflicts:
#	src/ansys/aedt/toolkits/magnet_segmentation/backend/run_backend.py
#	src/ansys/aedt/toolkits/magnet_segmentation/ui/actions.py
#	src/ansys/aedt/toolkits/magnet_segmentation/ui/common/frontend_ui.py
#	src/ansys/aedt/toolkits/magnet_segmentation/ui/common/thread_manager.py
#	src/ansys/aedt/toolkits/magnet_segmentation/ui/common/toolkit.ui
@gmalinve gmalinve self-assigned this Mar 13, 2024
@github-actions github-actions bot added documentation Improvements or additions to documentation maintenance Package and maintenance related labels Mar 13, 2024
@wiz-inc-572fc38784
Copy link

wiz-inc-572fc38784 bot commented Mar 14, 2024

Wiz Scan Summary

IaC Misconfigurations 0C 0H 0M 0L 0I
Vulnerabilities 0C 0H 0M 0L 0I
Sensitive Data 0C 0H 0M 0L 0I
Total 0C 0H 0M 0L 0I
Secrets 0🔑

@gmalinve gmalinve marked this pull request as ready for review April 24, 2024 15:04
@SMoraisAnsys
Copy link
Collaborator

We'll probably have to wait for a new release to be created to be sure that the tests are passing correctly.
The PR that should fix the issue is ansys/pyaedt#4603

pyproject.toml Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
SMoraisAnsys
SMoraisAnsys previously approved these changes May 7, 2024
@SMoraisAnsys
Copy link
Collaborator

@gmalinve Seems like there is still a problem as we are facing the following issue

image

I won't have time to look into it before going into days off + ooo but feel free to contact me if I can help (or have a look)

@SMoraisAnsys
Copy link
Collaborator

@gmalinve I took the liberty of updating the README to fix the build package issue

SMoraisAnsys
SMoraisAnsys previously approved these changes May 10, 2024
Copy link
Member

@Samuelopez-ansys Samuelopez-ansys left a comment

Choose a reason for hiding this comment

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

Good job! Next week I will test the new release with the toolkit manager.

@gmalinve gmalinve merged commit 5720cef into main May 13, 2024
21 checks passed
@gmalinve gmalinve deleted the migration branch May 13, 2024 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation maintenance Package and maintenance related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants