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

Improve syntax of process_id property in the Desktop() class #5611

Open
Devin-Crawford opened this issue Dec 27, 2024 · 0 comments
Open

Improve syntax of process_id property in the Desktop() class #5611

Devin-Crawford opened this issue Dec 27, 2024 · 0 comments
Assignees
Labels
enhancement New features or code improvements

Comments

@Devin-Crawford
Copy link
Contributor

Description of the current limitations

The property name aedt_process_id should be renamed to process_id. The prefix aedt is redundant and unnecessary since the process_id property is associated with the Desktop() class.

Code sample expected

Add aedt_process_id with a
@property wrapper and deprecation warning. Replace all instances of the property with process_id.

Useful links and references

https://github.com/ansys/pyaedt/blob/main/src/ansys/aedt/core/desktop.py

@Devin-Crawford Devin-Crawford added the enhancement New features or code improvements label Dec 27, 2024
@Devin-Crawford Devin-Crawford self-assigned this Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New features or code improvements
Projects
None yet
Development

No branches or pull requests

1 participant