Skip to content

782 simple model to calculate the allowable current density in a superconducting coil#3116

Merged
timothy-nunn merged 3 commits intomainfrom
782-simple-model-to-calculate-the-allowable-current-density-in-a-superconducting-coil
Mar 27, 2024
Merged

782 simple model to calculate the allowable current density in a superconducting coil#3116
timothy-nunn merged 3 commits intomainfrom
782-simple-model-to-calculate-the-allowable-current-density-in-a-superconducting-coil

Conversation

@kj5248
Copy link
Copy Markdown
Collaborator

@kj5248 kj5248 commented Mar 26, 2024

Closes #782.

Removed previously commented out sections of code relating to the tfc_model which is no longer used (unused in over 4 years). Removed mention of tfc_model in error relating to stellarators, and marked error 104 as obsolete (error to do with tfc_model).

@timothy-nunn timothy-nunn requested review from mkovari and removed request for mkovari March 27, 2024 09:44
@timothy-nunn
Copy link
Copy Markdown
Collaborator

timothy-nunn commented Mar 27, 2024

Michael, does this remove everything the original issue referred to?

@mkovari
Copy link
Copy Markdown
Collaborator

mkovari commented Mar 27, 2024

Yes, this seems to close out the original issue. It seems that the Subroutine sctfjalw was removed a long time ago - this pull request just tidies up some code that was commented out anyway.

@timothy-nunn timothy-nunn merged commit cd33ed3 into main Mar 27, 2024
@timothy-nunn timothy-nunn deleted the 782-simple-model-to-calculate-the-allowable-current-density-in-a-superconducting-coil branch March 27, 2024 10:06
chris-ashe pushed a commit that referenced this pull request Apr 22, 2024
* removed unused commented out sections for tfc_model

* remove mention of tfc_model in error message if using thkcas with istell=1

* set unused tfc_model error as obsolete
chris-ashe pushed a commit that referenced this pull request May 1, 2024
* removed unused commented out sections for tfc_model

* remove mention of tfc_model in error message if using thkcas with istell=1

* set unused tfc_model error as obsolete
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.

"Simple model to calculate the allowable current density in a superconducting coil"

3 participants