Skip to content

Commit

Permalink
REFACTOR: Fields init arg name
Browse files Browse the repository at this point in the history
The error would probably not have appeared if
the __iniy__ method of Fields had "post_app" as input
argument instead of "app".
I assume this led to the confusion with providing an AEDT app
such as Hfss, ...
  • Loading branch information
SMoraisAnsys committed Nov 15, 2024
1 parent a5b5efb commit 53b99da
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions src/ansys/aedt/core/visualization/report/field.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,10 +67,13 @@ def _context(self):


class Fields(CommonReport):
"""Provides for managing fields."""
"""Handler to manage fields."""

def __init__(self, app, report_category, setup_name, expressions=None):
CommonReport.__init__(self, app, report_category, setup_name, expressions)
@pyaedt_function_handler(

Check warning on line 72 in src/ansys/aedt/core/visualization/report/field.py

View check run for this annotation

Codecov / codecov/patch

src/ansys/aedt/core/visualization/report/field.py#L72

Added line #L72 was not covered by tests
app="post_app",
)
def __init__(self, post_app, report_category, setup_name, expressions=None):
CommonReport.__init__(self, post_app, report_category, setup_name, expressions)

Check warning on line 76 in src/ansys/aedt/core/visualization/report/field.py

View check run for this annotation

Codecov / codecov/patch

src/ansys/aedt/core/visualization/report/field.py#L75-L76

Added lines #L75 - L76 were not covered by tests
self.domain = "Sweep"
self.primary_sweep = "Distance"

Expand Down

0 comments on commit 53b99da

Please sign in to comment.