Sergey Kosukhin authored 5 months ago and Yen-Chen Chen committed 5 months ago

This removes an anti-pattern from `CMakeLists.txt` (I'm very sorry for introducing it). The most annoying part about it is that it makes it difficult for a parent project (e.g. ICON) to support multi-configuration backends properly. Basically, the only right place to set `CMAKE_BUILD_TYPE` is on the command line when calling `cmake` (and generating for a single-configuration backend). Also, see [here](https://stackoverflow.com/questions/24460486/cmake-build-type-is-not-being-used-in-cmakelists-txt/24470998#24470998

).

Approved-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: bugfix

The present check in the `set_acc_host_or_device` is heavily used in ICON and cannot be changed easily.
This problem comes from the discussion at https://gitlab.dkrz.de/icon/icon-mpim/-/merge_requests/563#note_276277


Revert the changes.

Approved-by: Marek Jacob <1129-b380572@users.noreply.gitlab.dkrz.de>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: bugfix

Claudia Frauen authored 5 months ago and Yen-Chen Chen committed 5 months ago

Workaround for the CCE 17 requiring explicit copy for reduction variables (!99)

CCE 17 needed for AMD GPUs on LUMI now requires an explicit copy of reduction variables.
Add copy clauses for the reduction variables.

Approved-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: bugfix

Daniel Reinert authored 5 months ago and Yen-Chen Chen committed 5 months ago

The subroutines `init_zero_4d_[dp,sp,i4]` do not vectorize properly on NEC SX AURORA.

Loop collapsing is enforced by a compiler directive for the subroutines `init_zero_4d_[dp,sp,i4]`, in order to ensure proper vectorization on NEC SX AURORA.

Approved-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: feature

Pradipta Samanta authored 7 months ago and Yen-Chen Chen committed 7 months ago

Fixed installation; other projects can now find module files from fortran-support (!95)

## What is the bug
Any projects dependent on fortran-support are not able to find the associated module files of when the fortran-support is installed in the system
## How do you fix it
`CMAKE_INSTALL_PREFIX` was not used to define `INSTALL_INTERFACE`. I corrected it now.

Approved-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: bugfix

## What is the new feature
Move random number generator to `libmath-support`. (Adds in https://gitlab.dkrz.de/icon-libraries/libmath-support/-/merge_requests/33

)
## How is it implemented
The random number generator module is more suitable to be put into `libmath-support`. Thus we are moving it from `libfortran-support`.

**The next versions `libfortran-support` 2.0.0 and `libmath-support` 1.1.0 should be merged at the same time.**

Approved-by: Daniel Hupp <daniel.hupp@meteoswiss.ch>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: feature

Sergey Kosukhin authored 9 months ago and Yen-Chen Chen committed 9 months ago

This updates the `add_icon_c_test` macro in `cmake/gtest_helper.cmake`:
1. The macro is turned into a function to avoid namespace pollution.
2. The function is renamed to `fs_add_c_test`.
3. The function now accepts named options `SOURCES` (followed by a list of source files of the test) and `ARGS` (followed by a list of command-line arguments for the test).
4. The old signature of the macro is still supported, i.e. it is possible to call something like `add_c_test(UtilHashTest ctest_util_hash.cpp)`, but it made sense to me to switch to the new one.
5. The `UtilFileTest` test is now provided with an extra command-line argument and it runs on `${CMAKE_CURRENT_BINARY_DIR}` instead of `.` as it was before !28.

Co-authored-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Approved-by: Terry Cojean <terry.cojean@kit.edu>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: other

## What is the new feature
Add a public interface for `libfortran-support`. This is related to #9 
## How is it implemented
Add a `fortran_support.F90` collecting all subroutines and functions in the `libfortran-support` library.  
Now one can do `USE fortran_support` to access all public subroutines/functions in the library.

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: feature

## What is the bug
A package-NOTFOUND is undetected and does not raise a problem. See #20
This MR closes #20
## How do you fix it
The OpenMP find package is made `REQUIRED`.  
The find package for OpenACC has to stay `QUIET` or else it will try to look for both `OpenACC_C` and `OpenACC_Fortran`. (LUMI does not have `OpenACC_C` at the moment.)

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Approved-by: Sergey Kosukhin <sergey.kosukhin@mpimet.mpg.de>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: other

## What is the new feature
Make `lacc` mandatory.
## How is it implemented
With the changes in https://gitlab.dkrz.de/icon/icon-mpim/-/merge_requests/342, https://gitlab.dkrz.de/art/art/-/merge_requests/96, and https://gitlab.dkrz.de/jsbach/jsbach/-/merge_requests/165

, the `lacc` argument in all `fortran_tools` functions should be added explicitly.  
This MR makes sure that in the future, an error will pop up if `lacc` is not passed.

Approved-by: Marek Jacob <1129-b380572@users.noreply.gitlab.dkrz.de>
Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: other

## What is the bug
Running pipelines on images cannot do tests with OpenACC.
## How do you fix it
Change the CI configuration to using the `levante` shared runner. This runner submits jobs on Levante directly with the CI script.  
`Check typos` CI is also fixed by specifying the conda channel to search for the package.

Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: bugfix

## What is the new feature
Add a formatter for CMake files. And CI task for CMake formatting is also added.

## How is it implemented
We use the opensource tool https://github.com/cheshirekow/cmake_format

 for formatting.

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: feature

Roland Wirth authored 11 months ago and Yen-Chen Chen committed 11 months ago

String lists grow dynamically, removing the need for preallocated arrays. This simplifies
handling of large variable groups in the icon initialization phase.

The output lists of `add_to_list` are now `ALLOCATABLE`. They may be passed an unallocated
argument, which is treated like an empty list and allocated using the new `new_list`
subroutine. That subroutine initially allocates space for 8 strings. Every time the list
needs to grow, the allocated size doubles. This ensures constant amortized runtime for
adding to the list. No provisions are made for shrinking the list size, because they are
usually short-lived.

The tests for `add_to_list` are adapted for the new interface, checking list expansion.
A new test for `new_list` is added.

Approved-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: feature

## What is the bug
`mo_fortran_tools` was updated during the release preparation MR.

Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: bugfix

## What is the new feature
Improve README.

Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: feature

## What is the bug
FortUTF did not return correct error code for failed test.
## How do you fix it
[This PR(merged)](https://github.com/artemis-beta/FortUTF/pull/22

) fixed the problem.

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: bugfix

Daniel Hupp authored 1 year ago and Yen-Chen Chen committed 1 year ago

## What is the new feature
This PR adds a new module for random number generations. It contains a parallelization uniform random number generator and a normal random number generator.
## How is it implemented
The uniform random number generator uses the Lehmer random number generator.
The normal random number generator uses a Box-Mueller transform using the uniform random number generator.

Approved-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: feature

<!--
ICON
## What is the bug
LUMI does not have `OpenACC_C` and the BB fails because we are checking `OpenACC` for both `C` and `Fortran`. This MR fixes `lumi_gpu` BB.
## How do you fix it
Only check if `OpenACC_Fortran` is found.

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: bugfix

## What is the new feature
Add a typo check pipeline.
## How is it implemented
Use https://github.com/crate-ci/typos



Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: feature

## What is the new feature
Make `lacc` arguments mandatory. This MR closes #17



Approved-by: Marek Jacob <1129-b380572@users.noreply.gitlab.dkrz.de>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: bugfix

## What is the bug
Fix a test for the if_associated subroutine.
## How do you fix it
The false test did not pop up because of a bug in FortUTF. See #18



Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: bugfix

## What is the bug
The accuracy of the expression evaluation code was falsely downscaled in the code.
## How do you fix it
The false test did not pop up because of a bug in FortUTF. See #18



Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: bugfix

## What is the new feature
Add tests
## How is it implemented
Add tests in `mo_expression` and the test coverage badge has officially turned yellow. 



Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: feature

Jonas Jucker authored 1 year ago and Yen-Chen Chen committed 1 year ago

New instructions for the merge request workflow.

Approved-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: other

## What is the bug
Fix a bug in unit test ([here](!79 (diffs))

) and improve code coverage
## How do you fix it
The code coverage is improved to 100% for `mo_util_sort`, and new tests with random values are added.

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: bugfix

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the new feature
Add OpenACC beautifier
## How is it implemented
The OpenACC beautifier used in icon [here](https://gitlab.dkrz.de/icon/icon-mpim/-/blob/master/.gitlab-ci.yml?ref_type=heads#L21-L55

)

## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] New feature is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [x] Reviewed by a maintainer
- [x] Incorporate review suggestions
- [x] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Co-authored-by: Marek Jacob <1129-b380572@users.noreply.gitlab.dkrz.de>
Approved-by: Marek Jacob <1129-b380572@users.noreply.gitlab.dkrz.de>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: feature

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the bug
The way of type conversion [here](https://gitlab.dkrz.de/icon-libraries/libfortran-support/-/blame/55913610ae02141850eb92c2ccc3f58759344646/src/mo_expression.F90?page=2#L1073

) is not suggested.
## How do you fix it
Convert types specifically. 

## How urgent is the bugfix
- [x] I need it as soon as possible
- [ ] I can wait for a couple of days
- [ ] None of my current codes is directly affected

## Mandatory steps before review
- [ ] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] Bugfix is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [ ] Test coverage does not decrease
- [ ] Reviewed by a maintainer
- [ ] Incorporate review suggestions
- [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Co-authored-by: Marek Jacob <1129-b380572@users.noreply.gitlab.dkrz.de>
Approved-by: Marek Jacob <1129-b380572@users.noreply.gitlab.dkrz.de>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: bugfix

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the bug
Tests in for loop are all the same.
## How do you fix it


## How urgent is the bugfix
- [ ] I need it as soon as possible
- [x] I can wait for a couple of days
- [ ] None of my current codes is directly affected

## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] Bugfix is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [ ] Reviewed by a maintainer
- [ ] Incorporate review suggestions
- [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: bugfix

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the new feature
Improve OpenACC configuration
## How is it implemented
This MR follows the configuration originally set up in the `math-support` library. See [here](https://gitlab.dkrz.de/icon-libraries/libmath-support/-/blob/6ad1577a20bbae01be6050bdb8263f39653cd348/cmake/check_macro.cmake) for the reference.  
~~This MR waits for !69

 to be merged first.~~

## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] New feature is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [x] Reviewed by a maintainer
- [x] Incorporate review suggestions
- [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Approved-by: Pradipta Samanta <samanta@dkrz.de>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: feature

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the new feature
_Please describe your feature in a couple of words._
## How is it implemented
_Describe important implementation details of the feature._

## Mandatory steps before review
- [ ] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] New feature is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [x] Reviewed by a maintainer
- [x] Incorporate review suggestions
- [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Approved-by: Pradipta Samanta <samanta@dkrz.de>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: feature

## What is the bug
Old CI tags are removed from Levante
## How do you fix it
Update CI tags


Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: bug

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the new feature
_Please describe your feature in a couple of words._
## How is it implemented
_Describe important implementation details of the feature._

## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] New feature is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [x] Reviewed by a maintainer
- [x] Incorporate review suggestions
- [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: other

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the new feature
Continue adding C tests. This MR closes #6


## How is it implemented


## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] New feature is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [x] Reviewed by a maintainer
- [x] Incorporate review suggestions
- [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Approved-by: Terry Cojean <terry.cojean@kit.edu>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: other

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the bug
The header check macros are always true
## How do you fix it
Make it configured to the same as the CMake variable, where the presence of the header is checked.

## How urgent is the bugfix
- [ ] I need it as soon as possible
- [x] I can wait for a couple of days
- [ ] None of my current codes is directly affected

## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] Bugfix is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [ ] Reviewed by a maintainer
- [ ] Incorporate review suggestions
- [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: bugfix

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the bug
Fix the problem that release is not created automatically on tag
## How do you fix it
Pipeline workflow was ruled out for tags

## How urgent is the bugfix
- [x] I need it as soon as possible
- [ ] I can wait for a couple of days
- [ ] None of my current codes is directly affected

## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] Bugfix is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [x] Reviewed by a maintainer
- [ ] Incorporate review suggestions
- [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: feature

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the new feature
This MR makes sure that the new version with OpenACC feature can be properly merged back to `icon-mpim`
## How is it implemented
See MR in icon-mpim for BuildBot results. All BuildBots must pass.
- Doxygen-style comments are removed to following the ICON style guide
- Intent specifiers are manually capitalized. This is an open issue for `fprettify`. See [here](https://github.com/pseewald/fprettify/issues/108

)
- [x] Put back missing subroutines

## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] New feature is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [x] Reviewed by a maintainer
- [x] Incorporate review suggestions
- [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: feature

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the new feature
Add unit tests for C functions
## How is it implemented
Use Googletest to implement missing unit tests in #6



## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] New feature is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [ ] Reviewed by a maintainer
- [ ] Incorporate review suggestions
- [ ] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: other

Jonas Jucker authored 1 year ago and Yen-Chen Chen committed 1 year ago

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the new feature
Add more unit-tests for `mo_util_string`
## How is it implemented
Each public subroutine has its own test

## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] New feature is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [x] Reviewed by a maintainer
- [x] Incorporate review suggestions
- [x] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Co-authored-by: Jonas Jucker <jonas.jucker@c2sm.ethz.ch>
Approved-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: other

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the new feature
_Please describe your feature in a couple of words._
## How is it implemented
This branch is a temporary solution to the requirement of `mo_fortran_tools` required for the math libraries.  
The adapted version of `mo_fortran_tools` removes the `i_am_accel_node` variable, therefore can only either not use OpenACC or use OpenACC for all nodes.

- [x] This branch uses the style from !17. Merge !17

 first.

## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] New feature is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Test coverage does not decrease
- [x] Reviewed by a maintainer
- [x] Incorporate review suggestions
- [x] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Co-authored-by: yanjen <yanjen224@gmail.com>
Co-authored-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Co-authored-by: Pradipta Samanta <samanta@dkrz.de>
Approved-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Merged-by: Yen-Chen Chen <yen-chen.chen@kit.edu>
Changelog: feature

<!--
ICON

---------------------------------------------------------------
Copyright (C) 2004-2024, DWD, MPI-M, DKRZ, KIT, ETH, MeteoSwiss
Contact information: icon-model.org

See AUTHORS.TXT for a list of authors
See LICENSES/ for license information
SPDX-License-Identifier: CC0-1.0
---------------------------------------------------------------
-->

## What is the new feature
Add badges
## How is it implemented
Adding to README. Pipelines for master branch has to be enabled for pipeline and coverage badges to show.
- [x] This MR has to wait for !57



## Mandatory steps before review
- [x] Gitlab CI passes _(Hint: use `make format` for linting)_ 
- [x] New feature is covered by additional unit tests
- [x] Mark the merge request as ready by removing `Draft:`

## Mandatory steps before merge
- [x] Reviewed by a maintainer
- [x] Incorporate review suggestions
- [x] Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)

**You are not supposed to merge this request by yourself, the maintainers of fortan-support take care of this action!**

Merged-by: Jonas Jucker <jonas.jucker@env.ethz.ch>
Changelog: other