Skip to content

Slight modification to copper fraction description within TF coil documentation#3343

Merged
chris-ashe merged 1 commit intomainfrom
jmatthews/1292-fcu-has-two-definitions
Oct 8, 2024
Merged

Slight modification to copper fraction description within TF coil documentation#3343
chris-ashe merged 1 commit intomainfrom
jmatthews/1292-fcu-has-two-definitions

Conversation

@kj5248
Copy link
Copy Markdown
Collaborator

@kj5248 kj5248 commented Oct 8, 2024

Description

Added to copper fraction description to mention that it does not include the copper in REBCO tape when using a HTS material.

Closes #1292.

@kj5248 kj5248 requested a review from chris-ashe October 8, 2024 08:40
@kj5248 kj5248 linked an issue Oct 8, 2024 that may be closed by this pull request
4 tasks
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 26.58%. Comparing base (7e15d39) to head (1b40db0).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3343   +/-   ##
=======================================
  Coverage   26.58%   26.58%           
=======================================
  Files          76       76           
  Lines       17606    17606           
=======================================
  Hits         4680     4680           
  Misses      12926    12926           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Collaborator

@chris-ashe chris-ashe left a comment

Choose a reason for hiding this comment

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

LGTM

@chris-ashe chris-ashe merged commit 13a927f into main Oct 8, 2024
@chris-ashe chris-ashe deleted the jmatthews/1292-fcu-has-two-definitions branch October 8, 2024 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fcu has two definitions

3 participants