Skip to content

Conversation

@fwlfHS
Copy link
Contributor

@fwlfHS fwlfHS commented Feb 4, 2026

Description

The HFSS Method create_linear_count_sweep takes the argument units, wheras create_linear_step_sweep and 3DLayout equivalent take the argument unit without the s.

https://aedt.docs.pyansys.com/version/stable/API/_autosummary/ansys.aedt.core.hfss.Hfss.create_linear_step_sweep.html
https://aedt.docs.pyansys.com/version/stable/API/_autosummary/ansys.aedt.core.hfss.Hfss.create_linear_count_sweep.html
https://aedt.docs.pyansys.com/version/stable/API/_autosummary/ansys.aedt.core.modules.solve_setup.SetupHFSS.create_frequency_sweep.html

In addition the typo in enable_adaptive_setup_broadband argument high_frequency has been corrected:
https://aedt.docs.pyansys.com/version/stable/API/_autosummary/ansys.aedt.core.modules.solve_setup.SetupHFSS.enable_adaptive_setup_broadband.html

Issue linked

--

Checklist

  • I have tested my changes locally.
  • I have added necessary documentation or updated existing documentation.
  • I have followed the coding style guidelines of this project.
  • I have added appropriate tests (unit, integration, system).
  • I have reviewed my changes before submitting this pull request.
  • I have linked the issue or issues that are solved by the PR if any.
  • I have agreed with the Contributor License Agreement (CLA).

@fwlfHS fwlfHS requested a review from a team as a code owner February 4, 2026 11:41
@ansys-cla-bot
Copy link

ansys-cla-bot bot commented Feb 4, 2026

All Contributor License Agreement (CLA) signatures have been captured successfully. Thanks for contributing!

@Samuelopez-ansys
Copy link
Member

Hi @fwlfHS ,

Did you fill the form here?

https://developer.ansys.com/form/cla-acceptance

then I will cherry pick your commits

@fwlfHS
Copy link
Contributor Author

fwlfHS commented Feb 9, 2026

Hi @fwlfHS ,

Did you fill the form here?

https://developer.ansys.com/form/cla-acceptance

then I will cherry pick your commits

Hi Samuel, yes I did:

All Contributor License Agreement (CLA) signatures have been captured successfully. Thanks for contributing!

Thank you!

@Samuelopez-ansys
Copy link
Member

@fwlfHS Thank for your contribution. I cherry-picked your commit, and I opened this PR:

#7217

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.

2 participants