Skip to content

Detailed plasma part 3#4099

Open
chris-ashe wants to merge 19 commits intomainfrom
detailed_plasma_part_3
Open

Detailed plasma part 3#4099
chris-ashe wants to merge 19 commits intomainfrom
detailed_plasma_part_3

Conversation

@chris-ashe
Copy link
Collaborator

@chris-ashe chris-ashe commented Feb 17, 2026

This pull request introduces several new plasma physics variables and expands the visualization capabilities for plasma profile plots. The main improvements are the addition of new variables for deuteron and triton Larmor radii, alpha particle velocities, Coulomb logarithms, and plasma frequencies, as well as new and enhanced plotting functions to visualize these profiles. The changes also reorganize subplot assignments in the main plotting routine to accommodate the new plots.

Additions and enhancements to plasma physics variables:

  • Added new variables in physics_variables.py for deuteron and triton toroidal Larmor radii, alpha particle thermal and birth velocities, deuteron-triton and electron-alpha Coulomb logarithms, deuteron plasma frequency, and upper hybrid frequency.
  • Updated initialization logic in init_physics_variables() to include the new variables.

Plotting and visualization improvements:

  • Added plot_larmor_radius_profile function to visualize deuteron and triton Larmor radius profiles, and integrated it into the main plot routine.
  • Enhanced plot_velocity_profile to plot alpha particle thermal velocity and birth velocity, including a new legend and line style for clarity.
  • Improved plot_electron_frequency_profile to show harmonics of the electron Larmor frequency and upper hybrid frequency, and added a secondary x-axis for physical radius.
  • Expanded plot_plasma_coloumb_logarithms to include deuteron-triton and electron-alpha Coulomb logarithm profiles, with new color coding for each interaction.

Plot layout and figure management:

  • Modified subplot assignments in main_plot and increased the number of figure pages to accommodate the new plots, ensuring each profile is displayed on its own subplot and maintaining clarity in the overall visualization.

Checklist

I confirm that I have completed the following checks:

  • My changes follow the PROCESS style guide
  • I have justified any large differences in the regression tests caused by this pull request in the comments.
  • I have added new tests where appropriate for the changes I have made.
  • If I have had to change any existing unit or integration tests, I have justified this change in the pull request comments.
  • If I have made documentation changes, I have checked they render correctly.
  • I have added documentation for my change, if appropriate.

@chris-ashe chris-ashe added the Physics Relating to the physics models label Feb 17, 2026
@chris-ashe chris-ashe marked this pull request as ready for review February 17, 2026 14:03
@chris-ashe chris-ashe requested a review from a team as a code owner February 17, 2026 14:03
Copy link
Collaborator

@timothy-nunn timothy-nunn left a comment

Choose a reason for hiding this comment

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

Unit test failure needs to be fixed before a proper review is conducted.

Could you also include the plots that have been added/changed

@timothy-nunn timothy-nunn self-assigned this Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Physics Relating to the physics models

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments