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

Fix ECE EMag twin builder export #413

Open
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

JackB-Ansys
Copy link
Collaborator

Usability updates and additional detail added, restructuring of the example

@github-actions github-actions bot added documentation Improvements or additions to documentation bug Something isn't working labels Oct 25, 2024
Copy link

codecov bot commented Oct 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.39%. Comparing base (65d2c52) to head (0c45319).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #413   +/-   ##
=======================================
  Coverage   86.39%   86.39%           
=======================================
  Files          21       21           
  Lines        2366     2366           
=======================================
  Hits         2044     2044           
  Misses        322      322           

@HusainAdam HusainAdam removed their request for review November 14, 2024 15:14
@HusainAdam
Copy link
Contributor

Removed myself as a reviewer

ravi-ansys
ravi-ansys previously approved these changes Nov 20, 2024
examples/links/ece_export_for_twinbuilder.py Outdated Show resolved Hide resolved
examples/links/ece_export_for_twinbuilder.py Outdated Show resolved Hide resolved
file_id = open(sml_file, "w")
file_id.write("%6s\r\n" % "MODELDEF ECER_Model1")
file_id.write("%6s\r\n" % "MODELDEF ECE_e8")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we get this from the mot file name, instead of hard coding? (Will need to be careful of non-alphanumeric characters in the mot file name, as these may not be valid in the SML, for example spaces, special characters or non-ascii characters)

@@ -554,7 +718,7 @@ def read_parameters(json_file):
file_id.write(
"%6s\r\n"
% (
"INTERN NDSRC PECER_Model1 N0:=GND,"
"INTERN NDSRC PECE_e8 N0:=GND,"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As above, could PECE_e8 be made less hard coded?

@@ -563,7 +727,7 @@ def read_parameters(json_file):
"%6s\r\n" % " ( QUANT:={ AM0.I, AM1.I, AM2.I },"
' SRC:={ isrc, isrc, isrc, isrc }, TableData:="\ '
)
file_id.write("%6s" % ".MODEL ECER_Model1_table pwl TABLE=(")
file_id.write("%6s" % ".MODEL ECE_e8_table pwl TABLE=(")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As above, could this be less hard coded?

examples/links/thermal_twinbuilder.py Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants