Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installation with cmake does not build kite-tools automatically #13

Open
sassy-crick opened this issue Jan 18, 2023 · 2 comments
Open

Comments

@sassy-crick
Copy link

The current CMakeLists.txt does not seem to build the KITE-Tools program automatically, which makes a scripted approach a bit more complex on a HPC cluster. Furthermore, the tools/CMakeLists.txt files does only seem to install KITE-tools but not the also required Python files kite.py, process_arpes.py and process_single_shot.py it seems.

It would be great if the root CMakeLists.txt could install both, KITE and KITE-Tools, including any required Python files, in one go. At least add an option so the KITE-Tools can be build by running cmake once.

I think that would greatly improve and makes it easier for users to install the software, and more in line what other projects are doing.

Thanks for looking into this.

@BertJorissen
Copy link
Member

We are currently looking into the possibility of combining the two programs in one CMake-file. The Python-script kite.py still needs to be linked to Python by hand.

There was an old release of KITE that was packaged in a PyPi-package. However, this version is outdated and it goes against the main idea of KITE: calculating large systems on a large multi-core system. The pre-processing step is only really meant to be run on a personal computer.

As KITE (and Pybinding) is still getting new updates, we are happy to receive feedback to create a better installation process or workflow.

@sassy-crick
Copy link
Author

Thanks for the feedback.
I think what might help is in the 'root CMakeList.txt file to add an option like for example DKITE_TOOLS=ON so it could be turned off (or on) during configuration time using something like that -DKITE_TOOLS=ON . I am not that familiar with CMake files, so don't take that literal.

I have also a PR open with EasyBuild and that seems to work from what I can see. I hope that helps a bit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants