Deltares / Deltares/ProbabilisticLibrary

clang build is not complete

Open
#301 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C++
Stars
8
Forks
1
Avg merge
4d 18h
Merged PRs (30d)
4

Description

Kind of request

None

Enhancement description

Recently we use clang for the code coverage build.

This build is not complete: it can't build a dynamic c-wrapper and on linux there are difficulties with OpenMP.

At the same time we run during the sonar upload configuration the Microsoft C compiler with OpenMP disabled. It is a little bit suprising that that works at all,

Use case

When fixing the clang build, we could add the python tests to the code coverage.

Additional context

No response

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the clang code-coverage build and the Sonar upload configuration mentioned in the issue. Investigate why the dynamic C wrapper cannot be built and how OpenMP behaves on Linux, then determine how the Python tests should join coverage. Done means the clang coverage build completes, including the wrapper and OpenMP configuration, with the Python tests included.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, python
Domain
build-system, ci-cd, testing-qa
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.