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.

Sign up for GitHub

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

Jump to bottom

Bug located in Hfss.create_setup() #5618

Open
2 tasks done
Devin-Crawford opened this issue Dec 28, 2024 · 0 comments · May be fixed by #5619
Open
2 tasks done

Bug located in Hfss.create_setup() #5618

Devin-Crawford opened this issue Dec 28, 2024 · 0 comments · May be fixed by #5619
Assignees
@Devin-Crawford
Labels
bug Something isn't working

Comments

@Devin-Crawford
Copy link
Contributor

Devin-Crawford commented Dec 28, 2024 •
edited
Loading

Before submitting the issue

  • I have searched among the existing issues
  • I am using a Python virtual environment

Description of the bug

The docstring states that the default type is "HFSSDrivenAuto" but the default type is actually None`. When Noneis then passed toself._create_setup()`` the default type is "HFSSDriven".

Note also that autosweep in the file core\modules\setup_templates.py used the
"LinearStep" sweep type which differs from the HFSS default. This should also be changed to match
the default values of

  • "RangeType": "LienarCount"
  • "RangeCount": 501

Steps To Reproduce

This is easy to reproduce. Just create an HFSS setup with no arguments.

Which Operating System are you using?

Windows

Which Python version are you using?

3.10

Installed packages

All dependencies are installed.
@Devin-Crawford Devin-Crawford added the bug Something isn't working label Dec 28, 2024
@Devin-Crawford Devin-Crawford self-assigned this Dec 28, 2024
Devin-Crawford added a commit that referenced this issue Dec 28, 2024
@Devin-Crawford
Correct error to HFSS Setup default type #5618
c8a030b 
\ansys\aedt\core\modules\setup_templates.py
- Change autosweep default from "LinearStep" to "LinearCount" to match the default behavior of HFSS.
- Use "RangeCount": 501

\ansys\aedt\core\hfss.py
- Update docstring errors for create_setup()

\ansys\aedt\core\application.py
- Change default setup from 1 to 0 for HFSS where appropriate.

\ansys\aedt\core\application\analysis.py
- Correct an error that would always use an auto-generated name for the setup.
@Devin-Crawford Devin-Crawford linked a pull request Dec 28, 2024 that will close this issue
Open
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Devin-Crawford Devin-Crawford

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

FIX: HFSS Setup error #5618 ansys/pyaedt
1 participant
@Devin-Crawford