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

Compile hycom oasis intel #219

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

Conversation

lilleannette
Copy link
Collaborator

No description provided.

@lilleannette lilleannette removed the request for review from caglartac November 24, 2023 16:14
Removed triver from blkdat
Removed triver from blkdat
#EXTRALIBS += -L${HOME}/local/fabm/hycom/lib -lfabm
EXTRALIBS = $(LIBPSMILE) -lfftw3 -lnetcdf -lnetcdff
# from fabm
CPPFLAGS += -D_FABM_ -I${HOME}/local/fabm/hycom/include

Choose a reason for hiding this comment

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

Line 50 should be commented out, so that the default does not require fabm

Copy link
Collaborator

Choose a reason for hiding this comment

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

added the fix, was tested on an different setup, it works but please have a look

EXTRALIBS = $(LIBPSMILE) -lfftw3 -lnetcdf -lnetcdff
# from fabm
CPPFLAGS += -D_FABM_ -I${HOME}/local/fabm/hycom/include
EXTRALIBS += -L${HOME}/local/fabm/hycom/lib -lfabm

Choose a reason for hiding this comment

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

Line 51 should be commented out, so that the default does not require fabm

Copy link
Collaborator

Choose a reason for hiding this comment

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

added the fix, was tested on an different setup, it works but please have a look

Copy link

@HR-polar HR-polar left a comment

Choose a reason for hiding this comment

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

The code currently works well but assumes that fabm is installed. Therefore, a change is required in Linux.fram.ifort_hycom so that CPPFLAGS and EXTRALIBS do not include fabm. Once these are commented out, the code compiles successfully with the default setting as without fabm. I have put a comment in the relevant place pointing to the two lines that should be commented.

Added the optional switch for ECOSMO
INCLUDES = -I${ESMF_MOD_DIR} -I${ESMF_DIR}include -I./CICE/rundir/compile
OASIS_DIR = /cluster/projects/nn9878k/guibou/NANUK/oa3-mct_iimpi2022a/
LIBPSMILE = $(OASIS_DIR)/lib/libpsmile.MPI1.a $(OASIS_DIR)/lib/libmct.a $(OASIS_DIR)/lib/libmpeu.a $(OASIS_DIR)/lib/libscrip.
a

Choose a reason for hiding this comment

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

The code seems to compile of with the extension "a" of "libscrip" on the next line (line 20) rather than at the end of $(OASIS_DIR)/lib/libscrip. , but it would be best to delete the newline command for clarity

@TillRasmussen
Copy link
Collaborator

There seem to be two conflicts that needs to be resolved.

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.

4 participants