KratosMultiphysics / Kratos

Kratos Multiphysics (A.K.A Kratos) is a framework for building parallel multi-disciplinary simulation software. Modularity, extensibility and HPC are the main objectives. Kratos has BSD license and is written in C++ with extensive Python interface.
https://kratosmultiphysics.github.io/Kratos/
Other
1.03k stars 245 forks source link

MKLSolversApplication #1316

Closed AndreasWinterstein closed 6 years ago

AndreasWinterstein commented 6 years ago

I wanted to open a discussion about the MKLSolverApplication. As far as I know, it is currently not working. Since the MKLPardiso solvers are included in the EigenSolversApplication now, I think we might discuss what should be done with the MKLSolversApplication.

loumalouomega commented 6 years ago

I tried to use the last version of the MKL libraries, but it was not possible to me to compile it. We can think about update it, but I know is a lot of work.

El 17 ene. 2018 10:31 a. m., "AndreasWinterstein" notifications@github.com escribió:

I wanted to open a discussion about the MKLSolverApplication. As far as I know, it is currently not working. Since the MKLPardiso solvers are included in the EigenSolversApplication now, I think we might discuss what should be done with the MKLSolversApplication.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/KratosMultiphysics/Kratos/issues/1316, or mute the thread https://github.com/notifications/unsubscribe-auth/ATEMgLtX_E53c4aqd9Zfp-otFR7fkzyFks5tLb3ugaJpZM4RhCjR .

AndreasWinterstein commented 6 years ago

The question is, if this is necessary/worth to do it, if we can do it by the EigenSolversApplication.

loumalouomega commented 6 years ago

Well, in that case I think @pooyan-dadvand and @RiccardoRossi can contribute more to the discussion than me

RiccardoRossi commented 6 years ago

Hi Andreas, Philip, the only thing is that this way we NEED eigen if we want to use MKL. Anyway we already clarified that the Eigen license allows it, so it is not a strong reason, also i don't use Pardiso since years.

Having said this, @pablobecker, i know that some of you guys are using the Pardiso. Could u find out if it would affect the team if we take out the MKLApplication?

cheers Riccardo

pooyan-dadvand commented 6 years ago

ping @pablobecker

AndreasWinterstein commented 6 years ago

ping @pablobecker

RiccardoRossi commented 6 years ago

@pablobecker, i know you contacted your colleagues. Since as i understand they are not getting feedback, we'll proceed from this if we get no feedback until the 9th of february

pablobecker commented 6 years ago

Ok

On 5 February 2018 at 16:30, Riccardo Rossi notifications@github.com wrote:

@pablobecker https://github.com/pablobecker, i know you contacted your colleagues. Since as i understand they are not getting feedback, we'll proceed from this if we get no feedback until the 9th of february

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/KratosMultiphysics/Kratos/issues/1316#issuecomment-363119112, or mute the thread https://github.com/notifications/unsubscribe-auth/AYf5id3wLgPEOAW1RNzVz2tLg5npV8lbks5tRx59gaJpZM4RhCjR .

philbucher commented 6 years ago

@RiccardoRossi do we go ahead and remove it?

I can make the PR

pablobecker commented 6 years ago

please go ahead