gridapapps / GridapGeosciences.jl

Gridap drivers for geoscience applications
MIT License
35 stars 3 forks source link

GridapGeosciences Parallelization #26

Closed amartinhuertas closed 2 years ago

amartinhuertas commented 2 years ago

This is working in progress, thus I open the PR in draft mode so that we may start familarizing with the work I am doing, discuss whatever is needed, etc.

The final goal of this PR is introduce the MPI-parallel version of GridapGeosciences.jl. Some overarching goals are:

This PR introduces a bunch of new major packages which are required to support distributed-memory parallelization, namely:

[Solved] Some of these are still under active development, with yet unreleased ongoing versions, so that we may have to point to these via the Manifest.toml file as usual.

codecov-commenter commented 2 years ago

Codecov Report

Merging #26 (034a430) into master (d1c7b26) will not change coverage. The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #26    +/-   ##
=======================================
  Coverage    0.00%   0.00%            
=======================================
  Files          17      18     +1     
  Lines        1056    1258   +202     
=======================================
- Misses       1056    1258   +202     
Impacted Files Coverage Δ
src/DiagnosticTools.jl 0.00% <0.00%> (ø)
src/Helpers.jl 0.00% <0.00%> (ø)
src/ShallowWaterExplicit.jl 0.00% <0.00%> (ø)
src/ShallowWaterIMEX.jl 0.00% <0.00%> (ø)
src/ShallowWaterRosenbrock.jl 0.00% <0.00%> (ø)
src/ShallowWaterThetaMethodFullNewton.jl 0.00% <0.00%> (ø)
src/ThermalShallowWaterExplicit.jl 0.00% <0.00%> (ø)
src/mpi/CubedSphereDistributedDiscreteModels.jl 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update d1c7b26...034a430. Read the comment docs.

amartinhuertas commented 2 years ago

Some of these are still under active development, with yet unreleased ongoing versions, so that we may have to point to these via the Manifest.toml file as usual.

This has been fixed in 02630c7. Let us see if the tests pass.

amartinhuertas commented 2 years ago

Ok, tests are passing now! :muscle: Accepting PR !