dfnWorks icon indicating copy to clipboard operation
dfnWorks copied to clipboard

Easier installation suggestion

Open gbene opened this issue 4 months ago • 1 comments

Hi again!

I am opening another issue since it is completely unrelated from the first that I made today.

DFNWorks is fantastic but I find it a bit cumbersome to install and use pydfnworks. All the different components to compile locally in multiple machines and OSs takes quite a bit of time. I was wandering if you ever considered to organize and compact the pip package by including precompiled binaries of lagrit, DFGen etcetc. You could do this by including the binaries in a folder inside the package itself (so for example in pydfnworks>builds).

This would firstly simplify a lot the path management. Since the precompiled libraries are in a folder within the library there won't be the need to use the .dfnworksrc file. Secondly, with a bit of work, it could be possible to make DFNWorks crossplatform!

Implementation of this should not be that difficult because It would require to add the paths of the precompiled libraries in the package_data dict in setup.py like this:

        ...
        package_data={'': ['dfnGen/generation/output_report/figures/*png',
                           'builds/ConnectivityTest',
                           'builds/correct_stor',
                           'builds/correct_uge',
                           'builds/DFNGen',
                           'builds/lagrit',]}
      )

and then add in the init.py a way to get the internal path (that is also the the dfnworks_PATH). Something like this:

...
file_path = os.path.realpath(__file__)

internal_filepath = os.path.dirname(file_path)
os.environ['dfnworks_PATH'] = internal_filepath

os.environ['DFNGEN_EXE'] = os.path.join(os.environ['dfnworks_PATH'], 'builds/DFNGen')
os.environ['CORRECT_UGE_EXE'] = os.path.join(os.environ['dfnworks_PATH'], 'builds/correct_uge')
os.environ['CORRECT_STOR_EXE'] = os.path.join(os.environ['dfnworks_PATH'], 'builds/correct_stor')
os.environ['CONNECT_TEST_EXE'] = os.path.join(os.environ['dfnworks_PATH'], 'builds/ConnectivityTest')
os.environ['LAGRIT_EXE'] = os.path.join(os.environ['dfnworks_PATH'], 'builds/lagrit')
...

The difficult part would be to automate the compilation of the different libraries (maybe using github actions?).

Let me know your thoughts about this!

Thank you again!

gbene avatar Feb 16 '24 11:02 gbene