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

FEAT: Tansformer loss distribution extension #5818

Draft
wants to merge 26 commits into
base: main
Choose a base branch
from
Draft

FEAT: Tansformer loss distribution extension #5818

wants to merge 26 commits into from

Conversation

gmalinve
Copy link
Contributor

Description

Please provide a brief description of the changes made in this pull request.

Issue linked

Please mention the issue number or describe the problem this pull request addresses.

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).

@gmalinve gmalinve self-assigned this Feb 23, 2025
@ansys-reviewer-bot
Copy link
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@gmalinve gmalinve changed the title FEAT: tTansformer loss distribution extension FEAT: Tansformer loss distribution extension Feb 23, 2025
@gmalinve gmalinve linked an issue Feb 23, 2025 that may be closed by this pull request
Maxwell: Export of transformer loss distribution extension #5807
Open
@github-actions github-actions bot added the enhancement New features or code improvements label Feb 23, 2025
@codecov Codecov
Copy link

codecov bot commented Feb 23, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 0% with 221 lines in your changes missing coverage. Please review.

Project coverage is 18.13%. Comparing base (38e0ec9) to head (b0e29de).
Report is 4 commits behind head on main.

Additional details and impacted files 
@@             Coverage Diff             @@
##             main    #5818       +/-   ##
===========================================
- Coverage   84.92%   18.13%   -66.80%     
===========================================
  Files         161      162        +1     
  Lines       62152    62375      +223     
===========================================
- Hits        52785    11310    -41475     
- Misses       9367    51065    +41698
🚀 New features to boost your workflow:
  • ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gmalinve gmalinve marked this pull request as ready for review February 28, 2025 14:49
@gmalinve gmalinve marked this pull request as draft March 6, 2025 17:15
gmalinve and others added 3 commits March 7, 2025 16:17
@gmalinve
fixes
b0e29de
@Devin-Crawford
Modify info message
03371f7 
- info message is a bit more polite.
@Devin-Crawford
Update for Field handling
a3d6a9a 
- Allow complex vector to be read in _read_mesh_files()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maxcapodi78 maxcapodi78 Awaiting requested review from maxcapodi78

@Alberto-DM Alberto-DM Awaiting requested review from Alberto-DM

@Samuelopez-ansys Samuelopez-ansys Awaiting requested review from Samuelopez-ansys

@SMoraisAnsys SMoraisAnsys Awaiting requested review from SMoraisAnsys

At least 1 approving review is required to merge this pull request.

Assignees

@gmalinve gmalinve

Labels
enhancement New features or code improvements
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Maxwell: Export of transformer loss distribution extension
2 participants
@gmalinve @Devin-Crawford