DARMA-tasking / LB-analysis-framework

Analysis framework for exploring, testing, and comparing load balancing strategies
Other
3 stars 1 forks source link

#376: rework import structure in lbafapp and other files #387

Closed tlamonthezie closed 1 year ago

tlamonthezie commented 1 year ago

Fixes #376

This pull request includes the following related to the rework of the import structure

Also includes changes from #367 following a merge done because of too many conflicts between the 2 branches

ppebay commented 1 year ago

@tlamonthezie when trying to run python LBAF_app.py --config conf.yaml for this branch, I am getting the following error:

Traceback (most recent call last):
  File "LBAF_app.py", line 11, in <module>
    from lbaf import __version__, PROJECT_PATH
ModuleNotFoundError: No module named 'lbaf'
tlamonthezie commented 1 year ago

Hello @ppebay it seems that your python path is not valid or you did not install the updated requirements file

ppebay commented 1 year ago

@tlamonthezie I am not sure what you mean by the fact that my Python path not being valid, as I have not changed anything, and am able to run LBAF develop without any issue. It is only with this PR that I am encountering the problem described above.

tlamonthezie commented 1 year ago

@tlamonthezie I am not sure what you mean by the fact that my Python path not being valid, as I have not changed anything, and am able to run LBAF develop without any issue. It is only with this PR that I am encountering the problem described above.

@ppebay requirements installation is enough to run LBAF. Could you please run the command to install the requirements again and let me know if it works on your side. In the CI and on my machine it works correctly. The way we run LBAF with develop is no more applicable with this PR because the full logic of imports has been changed. If you install the requirements you do not have to change your Python path (lbaf is automaticcally available as python package so no need to add to the Python path)

ppebay commented 1 year ago

@tlamonthezie now I understand! I thought my packages were up-to-date (for 3.8) but with the latest changes it's not the case any more. Now that makes more sense like this. Thank you.

I was able to run, and I am now verifying, test case by test case.

Stay tuned...

ppebay commented 1 year ago

Everything working as expected.

Thank you for the hard work @tlamonthezie