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

FC5PM compset fails to buil on Titan with PGI compiler #263

Closed
acme-y9s opened this issue Jul 6, 2015 · 10 comments
Closed

FC5PM compset fails to buil on Titan with PGI compiler #263

acme-y9s opened this issue Jul 6, 2015 · 10 comments

Comments

@acme-y9s
Copy link
Contributor

acme-y9s commented Jul 6, 2015

I tried to run ACME with the prescribed aerosol mode on Titan, but failed to built it with PGI compiler.
The last several lines from atm bld log file are
PGF90-F-0004-Unable to open MODULE file modal_aero_convproc.mod (/lustre/atlas1/cli112/proj-shared/y9s/ACME/models/atm/cam/src/physics/cam/physpkg.F90: 1747)
PGF90/x86-64 Linux 14.10-0: compilation aborted
gmake: *** [physpkg.o] Error 2
gmake: *** Waiting for unfinished jobs....

Who has succeeded in running this compset on Titan or other platforms?

@worleyph
Copy link
Contributor

worleyph commented Jul 6, 2015

This is the same bug as #227, and is being addressed as part of PR #243 . This is currently in the integration phase, so should be complete soon?

@singhbalwinder
Copy link
Contributor

Hi @acme-y9s ,
This has been fixed in singhbalwinder/atm/polar-mods-refactoring branch. This branch is yet to be merged but you can download this code to see if it works for you.

@acme-y9s
Copy link
Contributor Author

acme-y9s commented Jul 6, 2015

Thank you, I will try and let you know the result.

@acme-y9s
Copy link
Contributor Author

acme-y9s commented Jul 6, 2015

I tried to checked out the branch, but failed by the below two command lines.
git branch singhbalwinder/atm/polar-mods-refactoring --track origin/singhbalwinder/atm/polar-mods-refactoring

git checkout singhbalwinder/atm/polar-mods-refactoring

No response for the second command line.

Maybe, I do not have the permission ?

@singhbalwinder
Copy link
Contributor

Not sure what is going on here. I can checkout this branch just fine. @jedbrown , @douglasjacobsen , @jgfouca , do you guys know what might be causing this?

@douglasjacobsen
Copy link
Member

@acme-y9s: What does git log -n 1 output?

@acme-y9s
Copy link
Contributor Author

acme-y9s commented Jul 7, 2015

@singhbalwinder: I can successfully build both fc5pm compset and CAM4+CLM4.5 on Titan with your branch.
However, i failed to run the CAM4+CLM4.5 due to the urban net long wave radiation error ( no convergence). For the CAM5 + CLM4.5 case, it still on queue due to the computing nodes are not working today on Titan. Will let you know whether CAM5+CLM4.5 works or not.

@bishtgautam
Copy link
Contributor

@acme-y9s : @worleyph had reported a similar issue ("urban net longwave radiation error: no convergence") with the PGI a while back (see #165). Does this case runs successfully with intel on Titan?

@bishtgautam
Copy link
Contributor

@acme-y9s : After a little bit more digging, I realize that singhbalwinder/atm/polar-mods-refactoring branch was forked off before PR #178, which fixes the urban net longwave radiation, was merged into master. After PR #243 is merged in mater, CAM(4/5) + CLM4.5 should work correctly with PGI on Titan.

@acme-y9s
Copy link
Contributor Author

acme-y9s commented Jul 7, 2015

Thanks Gautam!


Xiaoying Shi
Climate Change Science Institute
Environmental Sciences Division
Building 2040, Room E222, MS-6301
Oak Ridge National Laboratory
P.O. Box 2008
Oak Ridge, TN 37831-6301
Office: 865-241-9199 Mobile: 865-804-1900
shix@ornl.govmailto:maoj@ornl.gov 865-574-9501 (fax)
[cid:0C731E85-6831-431B-9BAD-F7270C06BF7F]

From: Gautam Bisht <notifications@github.commailto:notifications@github.com>
Reply-To: ACME-Climate/ACME <reply@reply.github.commailto:reply@reply.github.com>
Date: Tuesday, July 7, 2015 at 11:11 AM
To: ACME-Climate/ACME <ACME@noreply.github.commailto:ACME@noreply.github.com>
Cc: Xiaoying Shi <shix@ornl.govmailto:shix@ornl.gov>
Subject: Re: [ACME] FC5PM compset fails to buil on Titan with PGI compiler (#263)

@acme-y9shttps://github.com/acme-y9s : After a little bit more digging, I realize that singhbalwinder/atm/polar-mods-refactoring branch was forked off before PR #178#178, which fixes the urban net longwave radiation, was merged into master. After PR #243#243 is merged in mater, CAM+CLM4.5 should work correctly with PGI on Titan.

Reply to this email directly or view it on GitHubhttps://github.com//issues/263#issuecomment-119234513.

mt5555 pushed a commit that referenced this issue Jul 10, 2015
This commit refactors unified convection changes so that MAM
specific changes in physpkg.F90 are moved to checmistry/modal_aero
directory. This will enable non-MAM acme configurations (such as
prescribed aerosols and CAM4) to function. Currently, the code fails
when run in prescribed aerosol mode.I have tested the prescribed
aerosol configuration and the code compiles and runs fine for a
few time step test case.

The commit also fixes three bugs found while running SE dycore
(ne30) with FC5 compset.

Fixes #227
Fixes #263
PR #243
AG-269
[BFB]
@mt5555 mt5555 closed this as completed in 61acbde Jul 12, 2015
rljacob added a commit that referenced this issue Jul 25, 2016
51f1282 Merge branch 'rljacob/acmepe/use-big-list' (PR #285)
a14a31f Merge branch 'master' of github.com:ESMCI/cime
d518834 minor change for hobart only
605e942 Merge pull request #283 from ESMCI/mira_walltime_fix
5ef8068 Merge pull request #284 from mvertens/newtests
c552fc2 Use single pe layouts file.  Add mpas entries
462810e replaced ww3 with stub wave in several tests that are known to fail
a8b9e7c need to return default walltime
7075ff6 Merge branch 'master' of github.com:CESM-Development/cime
49e3c29 Minor fix
11d61ce Merge pull request #279 from ESMCI/jgfouca/cime_bisect
b15cf46 Upgrade cime_bisect
873856a Merge pull request #278 from jedwards4b/minor_fixes_for_cesm_port
b990f48 add code to cp testreporter script for cesm
6ad54e9 Merge pull request #277 from fischer-ncar/testreporter_fix
be6d1b3 both compare and generate baseline_action_codes are allowed
e76ddc1 Fix for testreporter on hobart.
49fb888 fix issue with _caseroot when cloning
6d29a93 Update testreporter to work with updated ESMCI CIME
b7ed88f Updating to current ESMCI cime master
dbf9a47 Merge pull request #276 from ESMCI/jayeshkrishna/get_mira_working_for_acme
b5a2ce0 Get Mira working with acme
7adda2e updates for cesm allactive tests and compsets
fea102e Merge pull request #263 from ESMCI/sarich/new_acme_config_pes
7e9de96 update config_pes for change from cam5 to cam6
0170f5a rename new xml file to config_pesall.xml until we can sort out which pes descriptions belong here
3edb28f add script for converting the config_pes xml file
72a34bd Add NO_BASELINE_STATUS for BFAIL
83c5b68 Merge branch 'master' of github.com:ESMCI/cime
8d781cf Merge remote-tracking branch 'jedwards/edison_port_fixes' (PR #274)
559ee29 Merge pull request #273 from ESMCI/rljacob/cime/acme-batch
dcf27f6 reduce defaults for driver tests (cesm only) check for pylint earlier
3274c2a Add batch type for all acme machines in config_batch
118bf20 fix calls to read_xml
592ef76 Merge pull request #270 from jedwards4b/mira-workflow
aeac91a cannot write case files if error happens before case is created
45b0eb1 Minor fix
8cb87ff Merge pull request #272 from ESMCI/jgfouca/fix_some_sky_errors
fa09dcb Fix some problems on skybridge
bb6f277 Revert "get correct task_count for each job"
652f9c3 get correct task_count for each job
a9f1679 workflow now working on mira
c94e293 pull from esmci cime5.0.0
ec07a4f remove some pes configurations that muddied things up (BGP, BGQ, TEST)
5eb86bc add in latest changes from acme repo
862335a Add converted acme config_pes.xml file
58c3820 Merge pull request #472 from mvertens/remove_esmf_interfaces
f624721 Removed all code related to using the ESMF interfaces in driver_cpl directory
800e517 removed esmf interfaces

git-subtree-dir: cime
git-subtree-split: 51f1282
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

5 participants