ikarus-project / ikarus

Ikarus is a C++-based library that uses DUNE modules to solve partial differential equations with the finite element method and more.
https://ikarus-project.github.io/
Other
4 stars 2 forks source link

Add GH Action for python package and remove setpath workaround #284

Closed henrij22 closed 3 months ago

henrij22 commented 3 months ago

I am building the complete package, build it with pip and then run all the tests.

⚠️ I disabled setpath for this I would argue that we can remove the whole setpath workaround and test the python stuff within vscodes devcontainers and potentially in devcontainers in clion when they are supported.

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.27%. Comparing base (1083552) to head (9b9a816).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #284 +/- ## ======================================= Coverage 85.27% 85.27% ======================================= Files 60 60 Lines 1983 1983 ======================================= Hits 1691 1691 Misses 292 292 ``` | [Flag](https://app.codecov.io/gh/ikarus-project/ikarus/pull/284/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ikarus-project) | Coverage Δ | | |---|---|---| | [tests](https://app.codecov.io/gh/ikarus-project/ikarus/pull/284/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ikarus-project) | `85.27% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ikarus-project#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

henrij22 commented 3 months ago

@rath3t I worked in your requested changes, seems to work as expected we should also add this as required? But i dont't know how. It unfortunatly runs slower than the normal Debian Python test