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.

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

Albedo pumping bugfix #67

Merged
merged 5 commits into from
Jan 9, 2024
Merged

Albedo pumping bugfix #67

merged 5 commits into from
Jan 9, 2024

Conversation

holm10
Copy link
Collaborator

@holm10 holm10 commented Jan 3, 2024

Addresses bug that affects albedo-like pumping at the plates

  • The pumped fraction is Gamma_pump = (1-alb)(nv^bar)/4)*A
  • Previously, A was taken as sx, the area of the cell normal to the poloidal direction (i.e. flux tube cross-section)
  • However, for shaped, non-orthogonal target plates, the projection results in a larger pumping area than the perpendicular, increasing pumping
  • This projection is now taken into account when isoldalbarea = 0
  • By default, isoldalbarea = 1, and can be varied continuously

Also fixes issues for Basis builds by meyer8.

Co-Authored-By: Thomas Rognlien [email protected]

llnl-fesp and others added 4 commits January 3, 2024 13:10
Addresses bug that affects albedo-like pumping at the plates
- The pumped fraction is Gamma_pump = (1-alb)*(n*v^bar)/4)*A
- Previously, A was taken as sx, the area of the cell normal to the poloidal direction (i.e. flux tube cross-section)
- However, for shaped, non-orthogonal target plates, the projection results in a larger pumping area than the perpendicular, increasing pumping
- This projection is now taken into account when isoldalbarea = 0
- By default, isoldalbarea = 1, and can be varied continuously

Co-Authored-By: Thomas Rognlien <[email protected]>
Copy link
Collaborator

@tdrognlien tdrognlien left a comment

Choose a reason for hiding this comment

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

I believe the changes made in defining and using the new poloidal-cell plate area areapl for albedo pumping of gas is correct. This area can be smoothly varied from the old incorrect value of com.sx to the correct value of com.sxnp by varying bbb.isoldalbarea from 1 to 0.

@holm10
Copy link
Collaborator Author

holm10 commented Jan 9, 2024

Changes checked manually by TDR, OK to override GitHub review process.

@holm10 holm10 merged commit 506a927 into LLNL:develop Jan 9, 2024
1 check passed
@holm10 holm10 deleted the albedo_area_bugfix branch January 9, 2024 23:32
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.

3 participants