-
Notifications
You must be signed in to change notification settings - Fork 142
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
Hfss argument refactoring #4481
Conversation
Delete deprecated methods with more than 2 0.x. releases
# Conflicts: # pyaedt/application/Analysis.py # pyaedt/application/Analysis3DLayout.py # pyaedt/hfss.py # pyaedt/icepak.py # pyaedt/maxwell.py # pyaedt/mechanical.py # pyaedt/modules/SolveSweeps.py # pyaedt/q3d.py # pyaedt/rmxprt.py
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes will greatly improve readability. It looks like there may have been some accidental changes to other argument names - not sure.
Co-authored-by: Kathy Pippert <[email protected]>
for more information, see https://pre-commit.ci
Co-authored-by: Kathy Pippert <[email protected]>
Co-authored-by: Kathy Pippert <[email protected]>
Co-authored-by: Kathy Pippert <[email protected]>
Co-authored-by: Kathy Pippert <[email protected]>
Co-authored-by: Kathy Pippert <[email protected]>
…hfss_argument_refactoring
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, really huge refactoring.
I left a minor comment to be complient with previous statement.
Notes:
- would it make sense to use f-strings here ? I can give you a hand if you want.
- great thing that some old deprecated methods got removed !
Co-authored-by: Devin <[email protected]>
Co-authored-by: Sébastien Morais <[email protected]>
@SMoraisAnsys Which cases do you refer to regarding f-strings? We can replace them, but maybe could be in a different pull request. |
Probably best to wait for an official end of support for Python 2.7 (IronPython). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
You are right @dcrawforAtAnsys, hopefully this will happen soon 😄 |
No description provided.