Georgiana Mania authored 1 month ago and Pradipta Samanta committed 1 month ago

Merged-by: Pradipta Samanta <samanta@dkrz.de>
Changelog: default

fixed a bug

fixed a bug

fixed a bug in testing

made it compile

made changes to debug the last version

Yen-Chen Chen authored 1 month ago and Pradipta Samanta committed 1 month ago

## What is the bug
The `nvhpc_gpu` manual test creates a new pipeline.
## How do you fix it
The manual job was created twice: once upon push and once by merge request event. Therefore, the second pipeline appears. Disabling the merge request event job fixes the CI duplication.

Approved-by: Pradipta Samanta <samanta@dkrz.de>
Merged-by: Pradipta Samanta <samanta@dkrz.de>
Changelog: bugfix

Pradipta Samanta authored 1 month ago and Yen-Chen Chen committed 1 month ago

Enabled the use of different partitions for CI jobs running on CPU and GPU (!47)

## What is the bug
CI jobs always ran on the shared partition, irrespective of whether they were on CPUs or GPUs
## How do you fix it
moved the place of defining the 'SCHEDULAR_PARAMETERS' 

Approved-by: Yen-Chen Chen <yen-chen.chen@tum.de>
Merged-by: Yen-Chen Chen <yen-chen.chen@tum.de>
Changelog: bugfix

Yen-Chen Chen authored 1 month ago and Pradipta Samanta committed 1 month ago

## What is the bug
1. CMake warning pops up when one uses FetchContent without setting the download extract time stamp or CMake Policy 0135.
2. `libfortran-support` has been made public, but we are still fetching it via ssh.
## How do you fix it
Specify CMake Policy 0135, same as [here in test/fortran](https://gitlab.dkrz.de/icon-libraries/libiconmath/-/blob/main/test/fortran/CMakeLists.txt?ref_type=heads#L13-15

), fetch `libfortran-support` with https.

Merged-by: Pradipta Samanta <samanta@dkrz.de>
Changelog: bugfix

Dylan Kierans authored 1 month ago and Pradipta Samanta committed 1 month ago

## What is the bug
This MR:
* Reverts all variables in `set_zlev()` back from double-precision to working-precision.
* Adds a single & double precision variants (previously working-precision) of `rotate_latlon()`

## Details
Reverting `set_zlev()` changes I introduced in !13

. This will only effect single-precision builds.

Co-authored-by: Pradipta Samanta <samanta@dkrz.de>
Merged-by: Pradipta Samanta <samanta@dkrz.de>
Changelog: feature

Pradipta Samanta authored 1 month ago and Yen-Chen Chen committed 1 month ago

## What is the bug
The CMake option `IM_ENABLE_OPENACC` was by mistake written as `MS_ENABLE_OPENACC` in `gitlab-ci.yml`
## How do you fix it
Using the correct option now

Approved-by: Yen-Chen Chen <yen-chen.chen@tum.de>
Merged-by: Yen-Chen Chen <yen-chen.chen@tum.de>
Changelog: bugfix

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

## What is the bug
The use of `c_int` as a variable name in the module `mo_lib_interpolation_scalar` would create conflicts during the C++ implementation of the code. This MR solves the issue #6


## How do you fix it
The variable name `c_int` is changed to `coeff_int`

Approved-by: Yen-Chen Chen <yen-chen.chen@tum.de>
Merged-by: Yen-Chen Chen <yen-chen.chen@tum.de>
Changelog: bugfix