Fortran good first issues

AI Summary: The user is questioning the implementation of Hirshfeld charge calculations in CP2K, specifically regarding the SHAPE_FUNCTION GAUSSIAN and SHAPE_FUNCTION DENSITY with SELF_CONSISTENT T methods. They believe these methods may not adhere to standard schemes and produce chemically unintuitive charges, seeking clarification and potential options to revert to older definitions.

AI Summary: This issue highlights a potential inaccuracy in how constrained molecular dynamics (cAIMD) is used to calculate potential of mean force (PMF) in some reported works. The user points out that these works incorrectly interpret the Lagrange multiplier from the SHAKE algorithm as the ensemble-averaged constrained force, neglecting important corrections (Z and G). While the official CP2K example is deemed acceptable for simple distance constraints, the user argues that for more complex collective variables (CVs), these corrections are crucial and their omission can lead to significant errors in derived activation energies and reaction rates.

AI Summary: This issue proposes renaming the `fill_threshold` parameter in EAMxx to `time_sampling_threshold` or a similar alternative. The goal is to avoid potential confusion with `horiz_remap_fill_threshold` and ensure clarity in the codebase.

AI Summary: The `no_create` parameter for GPU macros is implemented for OpenACC but causes an unconditional abort in the OpenMP directive generation. This breaks OpenACC-only builds because the OpenMP code is evaluated even when not needed, leading to a build failure.

AI Summary: This issue proposes to refactor the frequency change tolerance check in `need_check.f90` and `realloc_array`. Currently, the tolerance is hard-coded in two separate locations, leading to potential inconsistencies. The fix involves moving this tolerance value to a shared constants module to ensure it's updated uniformly across both subroutines.

AI Summary: The GCC Linux build test is failing due to a cache miss in the `cache-env` step, leading to the `install-dependencies-with-spack` step failing because the `SPACK_STACK_DIR` environment variable is not defined. The user attempted a fix by modifying `gcc.yml` but was unsuccessful, and the test has been temporarily disabled.

AI Summary: This issue proposes refactoring CP2K's Libint wrapper to address code duplication and improve extensibility. The current integration is spread across multiple files, making it challenging to add new electronic-integral APIs or alternative libraries. The discussion aims to explore a more unified interface and its organizational structure.

AI Summary: This issue proposes a significant enhancement to the CP2K documentation, specifically the Input Reference. The goal is to improve usability for new users by providing more detailed explanations of keywords, including practical usage, typical use cases, parameter sensitivity, and common pitfalls, rather than relying solely on brief descriptions and external literature. The suggestion is to expand theoretical background and computational methodology sections, cross-linking them with keyword descriptions, and potentially encouraging community contributions.

AI Summary: This issue proposes evaluating the migration of CP2K's internal Gaussian integral implementations to an external library like Libint. The goal is to potentially leverage existing Libint wrappers and improve efficiency or maintainability by consolidating integral calculations.

AI Summary: This issue proposes several small cleanup tasks across five tooling files. The majority involve removing duplicated code, such as identical result dictionaries, getter functions, and emoji maps. One task addresses hardcoded scheduling thresholds, and another identifies a redundant configuration dictionary.

AI Summary: The build system incorrectly assumes a target is fully configured if `CMakeCache.txt` exists, leading to issues when builds are interrupted. This can cause configure steps to be skipped permanently or dependencies to be incorrectly marked as built and installed, resulting in confusing build errors.

AI Summary: The `toolchain/modules` script silently truncates environment variable values containing spaces to their first word. This occurs because the `export` command is executed with an unquoted value, leading to word-splitting and subsequent errors that are hidden by output redirection. This can cause critical build configurations to be incomplete without any visible indication.

AI Summary: A bug in the case validator script causes an error message to reference a non-existent parameter `bc_{dir}%Tw_out` when checking for a positive isothermal wall temperature. The correct parameter name is `bc_{dir}%Twall_out`. This misdirects users when troubleshooting, although it does not affect the validation logic itself.

AI Summary: This issue describes a numerical instability in a fluid dynamics simulation involving moving Lagrangian bubbles with two-way coupling. The simulation results vary depending on the number of MPI ranks used, which is caused by subtle differences in floating-point summation order at subdomain boundaries. These small differences are amplified by the chaotic nature of the bubble dynamics, leading to significant trajectory divergence.