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

Pyaedt renderer #935

Merged
merged 28 commits into from
Apr 26, 2023
Merged

Pyaedt renderer #935

merged 28 commits into from
Apr 26, 2023

Conversation

priti-ashvin-shah-ibm
Copy link
Collaborator

@priti-ashvin-shah-ibm priti-ashvin-shah-ibm commented Apr 19, 2023

What are the issues this pull addresses (issue numbers / links)?

Since the ansys renderer requires paid full license and the use of Windows comm port, now, these renderers in the future MAY be used by student license which is free. (Presently the pull request has been tested with ONLY a paid full license.) Also, pyaedt API is used to connect to Ansys, so that we are not limited to windows comm-port. However, this has been only tested on widows OS.

The foundation has been built to use student licenses. It is an opportunity for people in the community to contribute in the future to test and fix bugs when using a student license for Ansys..

Did you add tests to cover your changes (yes/no)?

No

Did you update the documentation accordingly (yes/no)?

ues

Did you read the CONTRIBUTING document (yes/no)?

Summary

Details and comments

@priti-ashvin-shah-ibm
Copy link
Collaborator Author

As a test, I made a new conda env and then I ran the two notebooks on Ansys on windows.

@priti-ashvin-shah-ibm priti-ashvin-shah-ibm marked this pull request as ready for review April 20, 2023 03:12
@priti-ashvin-shah-ibm priti-ashvin-shah-ibm merged commit ea1ca0e into main Apr 26, 2023
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.

3 participants