Skip to content

152 PF coil#3036

Merged
jonmaddock merged 6 commits intomainfrom
152-pf-coil
Feb 6, 2024
Merged

152 PF coil#3036
jonmaddock merged 6 commits intomainfrom
152-pf-coil

Conversation

@ajpearcey
Copy link
Copy Markdown
Collaborator

closes #152

@ajpearcey ajpearcey changed the title 152 PF coil WIP: 152 PF coil Jan 29, 2024
@ajpearcey ajpearcey requested a review from jonmaddock February 5, 2024 10:01
Copy link
Copy Markdown
Contributor

@jonmaddock jonmaddock left a comment

Choose a reason for hiding this comment

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

Content is fine, but there's a bit of back-and-forth and formatting fixes (perhaps check pre-commit is working with black alright?). Can you rebase and fixup these commits to condense them to reflect only the required changes?

Comment thread source/fortran/pfcoil_variables.f90 Outdated
Comment thread source/fortran/iteration_variables.f90
@jonmaddock jonmaddock changed the title WIP: 152 PF coil 152 PF coil Feb 6, 2024
@jonmaddock jonmaddock merged commit bb5a94a into main Feb 6, 2024
@jonmaddock jonmaddock deleted the 152-pf-coil branch February 6, 2024 09:52
chris-ashe pushed a commit that referenced this pull request Apr 22, 2024
* ✨ add pf current safety factor

* 🎨 black formatting

* ✏️ update varialbe description

* 🎨 remove change to fdene bound

* 🐛 fix default SF value

* ✅ update large tokamak to pass regression test

---------

Co-authored-by: apearce <alexaner.pearce@ukaea.uk>
chris-ashe pushed a commit that referenced this pull request May 1, 2024
* ✨ add pf current safety factor

* 🎨 black formatting

* ✏️ update varialbe description

* 🎨 remove change to fdene bound

* 🐛 fix default SF value

* ✅ update large tokamak to pass regression test

---------

Co-authored-by: apearce <alexaner.pearce@ukaea.uk>
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.

PF coils

2 participants