Skip to content

Conversation

@AndresOrtegaGuerrero
Copy link

@AndresOrtegaGuerrero AndresOrtegaGuerrero commented Aug 21, 2025

Uenv recipe and Spack package for "wannier90"

@AndresOrtegaGuerrero
Copy link
Author

Dear @edoardob90 , I am a bit new to this , please find here the PR and maybe we can work on it ? , I didnt find the package recipe for wannier90, so I assumed that we needed to create a package as well.

Copy link
Member

@edoardob90 edoardob90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @AndresOrtegaGuerrero, looks good. A few changes before trying to build the uenv

@edoardob90 edoardob90 changed the title Uenv W90 and Package Wannier90 uenv and Spack package Aug 21, 2025
Copy link
Member

@edoardob90 edoardob90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AndresOrtegaGuerrero Seems all good to me. I made only a few reformatting changes. Before merging, I want to test the build first. Unfortunately, Daint.Alps computes nodes are still down today.

Lastly, I double-checked Spack's original recipe for "Wannier90". It contains several customizations to the compile and install steps. Are you sure that the package you created is not missing anything critical for the build to succeed?

@AndresOrtegaGuerrero
Copy link
Author

This part I dont know, as I mentioned in my email

F90 = gfortran
CC = cc

COMMS  = mpi
MPIF90 = mpif90

FCOPTS= -O3 -fPIC -g -fallow-argument-mismatch
LDOPTS= -O3 -fPIC

LIBS = -L/user-environment/env/default/lib  -lopenblas

was enough for the compilation in daint.alps. Besides that I am not sure

@edoardob90
Copy link
Member

@AndresOrtegaGuerrero I'm testing the uenv build on Daint. If that works fine, we can merge your PR

@AndresOrtegaGuerrero
Copy link
Author

@edoardob90 just let me know , thanks!

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

Successfully merging this pull request may close these issues.

2 participants