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

Improved openifs block #561

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

omula
Copy link
Contributor

@omula omula commented Feb 18, 2015

Hi,

I am trying to make openifs easyblock working with newer version and I've realised that FCM is like EasyBuild and that they provide several configurations to compile the model.

omula@login:/tmp/oifs38r1v04/make/cfg$ ls -1
cce-nansC.cfg
cce-noopt.cfg
cce-opt.cfg
gnu-nansC.cfg
gnu-noopt.cfg
gnu-opt.cfg
ibm-nansC.cfg
ibm-noopt.cfg
ibm-opt.cfg
intel_mkl-nansC.cfg
intel_mkl-noopt.cfg
intel_mkl-opt.cfg
intel-nansC.cfg
intel-noopt.cfg
intel-opt.cfg
lapack.cfg
oifs-depend.cfg
oifs_xlf_wrapper
pgi-nansC.cfg
pgi-noopt.cfg
pgi-opt.cfg
diff -rupN intel_mkl-opt.cfg intel_mkl-noopt.cfg
--- intel_mkl-opt.cfg 2014-10-13 16:25:37.000000000 +0200
+++ intel_mkl-noopt.cfg 2014-10-13 16:25:37.000000000 +0200
@@ -1,5 +1,5 @@
FCM compiler & build specific configuration file for OpenIFS

-# intel_mkl compiler, opt build
+# intel_mkl compiler, noopt build
Glenn Carver ECMWF 2012-2014

@@ -30,15 +30,15 @@ $OIFS_SRC_EXCL{?} =
Fortran

$OIFS_FC{?} = mpiifort
-$OIFS_FFLAGS{?} = -g -m64 -openmp -O1 -xHost -fp-model precise -convert big_endian -traceback
+$OIFS_FFLAGS{?} = -g -m64 -O0 -fp-model precise -convert big_endian -traceback
$OIFS_FFIXED{?} = -r8
$OIFS_FCDEFS{?} = BLAS LITTLE LINUX INTEGER_IS_INT
-$OIFS_LFLAGS{?} = -openmp
+$OIFS_LFLAGS{?} =
C compiler

$OIFS_CC{?} = mpiicc
-$OIFS_CFLAGS{?} = -g -O
+$OIFS_CFLAGS{?} = -g -O0
$OIFS_CCDEFS{?} = BLAS LITTLE LINUX INTEGER_IS_INT _ABI64
Architecture/compiler specific FCM (if any)

Further information on compiling OpenIFS can be found in https://software.ecmwf.int/wiki/display/OIFS/How+to+customize+compiling+OpenIFS

Which would be the best way to improve the easyblock? You can have a look to my current attempt now in https://github.com/omula/easybuild-easyblocks/tree/develop-openifs

In order to make it work I had to add the -convert big_endian to the flags. Would you leave it as it is right now?

Cheers,
Oriol

* Added support for multiple versions
@hpcugentbot
Copy link
Contributor

Automatic reply from Jenkins: Can I test this?

This was referenced Feb 18, 2015
@boegel
Copy link
Member

boegel commented Apr 26, 2015

Jenkins: ok to test

@boegel
Copy link
Member

boegel commented Apr 26, 2015

@wpoely86: up for giving some feedback on this?

@hpcugentbot
Copy link
Contributor

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/931/
Test PASSed.

@@ -48,11 +49,11 @@ def configure_step(self):

# configure build via OIFS_* environment variables
env.setvar('OIFS_ARCH', 'x86_64')
env.setvar('OIFS_BUILD', 'opt')
env.setvar('OIFS_BUILD', 'noopt')
Copy link
Member

Choose a reason for hiding this comment

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

why the noopt?

@boegel boegel modified the milestone: v2.2 Jun 24, 2015
@boegel boegel modified the milestones: v2.2, v2.3 Jul 7, 2015
@boegel boegel modified the milestones: v2.4.0, v2.5.0 Nov 6, 2015
@boegel boegel modified the milestones: v2.5.0, v2.6.0 Dec 14, 2015
@boegel
Copy link
Member

boegel commented Jan 23, 2016

@omula: ping?

@boegel boegel modified the milestones: v2.6.0, v2.7.0 Jan 23, 2016
@omula
Copy link
Contributor Author

omula commented Jan 26, 2016

@boegel: I have tested the easyblock with the next OpenIFS release 40r1 and it works.

@boegel
Copy link
Member

boegel commented Mar 9, 2016

@omula: is there a PR for an updated OpenIFS easyconfig file?

@boegel boegel added this to the v2.8.0 milestone Mar 9, 2016
@boegel boegel removed this from the v2.7.0 milestone Mar 9, 2016
@boegel boegel modified the milestones: v2.8.0, v2.x May 10, 2016
@boegel boegel modified the milestones: 3.x, 4.x Feb 20, 2020
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.

None yet

4 participants