Kratos: VisualStudio resets the libs folder in the KratosInstall

[AlejandroCornejo]

Hello!
I've pulled the master branch and I'm getting the following error regarding the ExternalSolversApplication:

Traceback (most recent call last):
File "KratosFemDemCoupledApplication.py", line 4, in
import CouplingFemDem
File "D:\KratosInstall\applications\FemToDemApplication\python_scripts\CouplingFemDem.py", line 4, in
import MainFEM_for_coupling as FEM
File "D:\KratosInstall\applications\FemToDemApplication\python_scripts\MainFEM_for_coupling.py", line 3, in
import MainFemDem
File "D:\KratosInstall\applications\FemToDemApplication\python_scripts\MainFemDem.py", line 10, in
import KratosMultiphysics.ExternalSolversApplication as KratosSolvers
File "D:\KratosInstall\KratosMultiphysics\ExternalSolversApplication__init__.py", line 3, in
from KratosExternalSolversApplication import *
ImportError: DLL load failed: No se puede encontrar el módulo especificado.
KRATOS TERMINATED WITH ERROR

Anyone knows what's happening?
Thank you!
@philbucher

[philbucher]

hm no clue...
For me it works, compiling master all the time
Also I am not too familiar with this app

Do you delete the old libs before recompiling?

If this doesn't work I recommend the following steps ;)

1. https://purplemonkeydishwasher.co.uk/2019/01/30/the-ubuntu-installation-guide/
2. https://github.com/KratosMultiphysics/Kratos/wiki/Linux-Build

[AlejandroCornejo]

It seems that recompiling everything works! thank you and sorry fot the inconvenience @philbucher

[AlejandroCornejo]

It is weird... The problem is with VisualStudio, when compiling the code it resets the libs folder and I have to copy the .dll everytime I compile Kratos! @roigcarlo Do you know it is performing in this way?
Thank you

[philbucher]

this happens after #4314

Reason is to remove possible "old" libraries e.g. if you no longer compile an application it should no longer be accessible

In windows you have additional libraries in this folder?

[AlejandroCornejo]

Yes! After the compilation we need to copy several files... It is explained
in the windows build in the wiki. Maybe we should review this

El mié., 6 mar. 2019 15:34, Philipp Bucher [e-mail]
escribió:

this happens after #4314
https://github.com/KratosMultiphysics/Kratos/pull/4314

Reason is to remove possible "old" libraries e.g. if you no longer compile
an application it should no longer be accessible

In windows you have additional libraries in this folder?

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/KratosMultiphysics/Kratos/issues/4356#issuecomment-470120091,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZLfLRy3EU4dj-z54LOjzUClMf6fiDxLks5vT9HugaJpZM4bgzgo
.

[roigcarlo]

Uh... yep. The changes we made to the libs folder will cause this. I will update the wiki now.

From now on if you don't want to copy the libs every time you will need to add the ming-gw and the folder with blas lapack to the system path.
https://github.com/KratosMultiphysics/Kratos/wiki/Windows-Install#post-compilation

Edit: @loumalouomega I've deleted the imagine because I want to to keep this one technical as many people could eventually have this problem

[philbucher]

we could add a "whitelist" when on Windows ...?

[roigcarlo]

We could, but people in will will have the problem with the old compiled libs again.
Or I could force cmake to install the requeired libs in the folder. I can do it without any changes for blas and lapack but I would need a new -DMINGW_DIRECTORY in the configure for the rest

[AlejandroCornejo]

thank you for your time! Shall i close the issue?

[philbucher]

configure

I meant you could delete everything except some libblas and lapack (and maybe some others, you can control what is on the whitelist)

[roigcarlo]

Leave it open until Friday just in case.

@philbucher I can do it for the recommended versions of the wiki, but if someone uses anything else will have the same problem :S

[philbucher]

ok I see
Well then at least it should be documented properly

[roigcarlo]

ok I see
Well then at least it should be documented properly

:+1: Already added in the wiki, but I will make a PR with the implementation of the whitelist solution

[loumalouomega]

Is this solved?

[roigcarlo]

Leave it open until Friday just in case.