Kratos icon indicating copy to clipboard operation
Kratos copied to clipboard

[Question] What is kratos_space.h?

Open loumalouomega opened this issue 3 years ago • 3 comments

Description

Today I realized that we have an implementation (newer and using parallel utilities instead of plain OMP) called kratos_space.h:

https://github.com/KratosMultiphysics/Kratos/blob/master/kratos/spaces/kratos_space.h

But this implementation is only used in the corresponding test, and is not called in anywhere except that place. I assume the idea is to replace the old and not up-to-date ublas_space.h:

https://github.com/KratosMultiphysics/Kratos/blob/master/kratos/spaces/ublas_space.h

But if the idea is that, why there is no PR replacing them? (https://github.com/KratosMultiphysics/Kratos/search?q=kratos_space.h+is%3Apr&type=issues). So I assume the idea is to remove the old ublas_space.h and replace it with kratos_space.h.

@philbucher and @RiccardoRossi do you know what is the status of this?

Scope

  • KratosCore
  • All applications using algebra

loumalouomega avatar Aug 05 '22 09:08 loumalouomega

it is the new one for the Kratos CSR-matrices

However most probably a new B&S is needed for it, which will then be also compatible with MPI

philbucher avatar Aug 05 '22 10:08 philbucher

it is the new one for the Kratos CSR-matrices

However most probably a new B&S is needed for it, which will then be also compatible with MPI

So, do we need to update the ublas_space.h to use ParallelUtilities?, or will we replace directly once the Kratos CSR-matrices is completed and therefore is meaningless to update it?

loumalouomega avatar Aug 05 '22 10:08 loumalouomega

in theory it will be eventually replaced, but this might take some more time hence I guess it can be updated if there is free capacity

philbucher avatar Aug 07 '22 19:08 philbucher