Skip to content

Conversation

@pvinod13
Copy link
Contributor

@pvinod13 pvinod13 commented Nov 5, 2025

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

pvinod13 and others added 14 commits November 5, 2025 12:38
- Modified assign_coil in maxwell.py to handle multiple object assignments as CoilGroup
- Updated test_assign_coil_group in test_27_Maxwell2D.py with comprehensive boundary counting validation
- Updated test_create_coil_group in test_28_Maxwell3D.py for 3D coil terminal groups
- Added proper boundary type validation and object assignment verification
@pvinod13 pvinod13 requested a review from a team as a code owner November 5, 2025 07:26
@github-actions github-actions bot added the enhancement New features or code improvements label Nov 5, 2025
@codecov
Copy link

codecov bot commented Nov 5, 2025

Codecov Report

❌ Patch coverage is 3.12500% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.12%. Comparing base (37233eb) to head (efd2fa7).

❌ Your patch check has failed because the patch coverage (3.12%) is below the target coverage (85.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #6855       +/-   ##
===========================================
+ Coverage   44.50%   57.12%   +12.61%     
===========================================
  Files         245      245               
  Lines       77613    77639       +26     
===========================================
+ Hits        34544    44348     +9804     
+ Misses      43069    33291     -9778     
🚀 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New features or code improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add AssignCoilGroup to be able to assign multiple coils to one group

4 participants