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

Clean up stella: kxky grids, namelists, debug flags, including full_flux_surface automatic numerical tests #156

Merged
merged 25 commits into from
Sep 26, 2024

Conversation

GeorgiaActon
Copy link
Collaborator

Continuation of clean up of stella. The kxky grids have been cleaned up, and separated out into different modules. There not exists a <parameters_kxky_grids.f90> file which is the overarching module that reads the parameters for the kxky grids. This then uses either <parameters_kxky_grids_box.f90> or <parameters_kxky_grids_range.f90> depending on which grid type is chosen in the input file. Two additional modules have been added as a result of this split up called <calculations_kxky.f90> and <write_radial_grid.f90>.

There is a new folder called <grids_kxky.f90> file that initialised the kxky grids and fills the arrays for the kxky grids.

The parameter namelist have also been changed, and placed into different modules. The new files are <parameters_numerical.f90> <parameters_physics.f90>, and have replaced the previous modules <run_parameters.f90>, <physics_flags.f90>, <physics_paramters.f90>. These have been named in preparation for organising stella into sub directories.

A new debug feature has also been added to allow the debug flags to be turned on/off in the input file, rather than requiring the code to be re-compiled. This is found in <debug_flags.f90>.

@GeorgiaActon GeorgiaActon changed the title Cleanup stella: kxky grids, namelists, debug flags, including full_flux_surface automatic numerical tests Clean up stella: kxky grids, namelists, debug flags, including full_flux_surface automatic numerical tests Sep 5, 2024
@HanneThienpondt HanneThienpondt linked an issue Sep 26, 2024 that may be closed by this pull request
@HanneThienpondt HanneThienpondt added clean-up Places where the code needs refactoring stella review 07/2024 Georgia and Hanne are cleaning up stella labels Sep 26, 2024
Copy link
Collaborator

@HanneThienpondt HanneThienpondt left a comment

Choose a reason for hiding this comment

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

Looks good! Note that the <flip_flop_old> variable is not initialized in parameters_physics.f90, but this bug is fixed in a later pull request.

@HanneThienpondt HanneThienpondt merged commit c0992ea into master Sep 26, 2024
5 checks passed
HanneThienpondt added a commit that referenced this pull request Sep 26, 2024
commit c0992ea
Author: GeorgiaActon <[email protected]>
Date:   Thu Sep 26 10:55:29 2024 +0100

    Clean up stella: kxky grids, namelists, debug flags, including full_flux_surface automatic numerical tests (#156)

    Continuation of clean up of stella. The kxky grids have been cleaned up, and separated out into different modules. There not exists a <parameters_kxky_grids.f90> file which is the overarching module that reads the parameters for the kxky grids. This then uses either <parameters_kxky_grids_box.f90> or <parameters_kxky_grids_range.f90> depending on which grid type is chosen in the input file. Two additional modules have been added as a result of this split up called <calculations_kxky.f90> and <write_radial_grid.f90>.

    There is a new folder called <grids_kxky.f90> file that initialised the kxky grids and fills the arrays for the kxky grids.

    The parameter namelist have also been changed, and placed into different modules. The new files are <parameters_numerical.f90> <parameters_physics.f90>, and have replaced the previous modules <run_parameters.f90>, <physics_flags.f90>, <physics_paramters.f90>. These have been named in preparation for organising stella into sub directories.

    A new debug feature has also been added to allow the debug flags to be turned on/off in the input file, rather than requiring the code to be re-compiled. This is found in <debug_flags.f90>.
Merge branch 'master' into New_cleanup_fields
This was linked to issues Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clean-up Places where the code needs refactoring stella review 07/2024 Georgia and Hanne are cleaning up stella
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add z-pinch geometry Add electromagnetic automated tests Store all debug flags in a seperate scripts.
2 participants