Corrects processor layout on Edison for ne16 grid #462
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I assume @ndkeen is Noel Keen. This is part of NERSC POC duties. If you are not comfortable integrating this to master following our Integration Guide, https://acme-climate.atlassian.net/wiki/display/Docs/Integrator+Guide, thats ok. We can assign it to someone else. |
|
Merged to next. Waiting for tests to run before merging to master. |
|
@ndkeen: Since the ERS.ne16_ne16.FC5MAM4.edison_intel was successful, this PR can now be merged into master. Thanks. |
amametjanov
added a commit
that referenced
this pull request
Dec 7, 2015
Makes the processor layout for each component on Edison be a multiple of 24 for ne16 grid. Fixes #461 [BFB] * bishtgautam/scripts/ne16-edison-test-fix: Corrects processor layout on Edison for ne16 grid Conflicts: cime/machines-acme/config_pes.xml
rljacob
added a commit
that referenced
this pull request
Jul 20, 2016
92a5d03 Merge pull request #271 from jedwards4b/misc_python_fixes 8659568 fix issues with code_checker and missed run_cmd changes 1f9f0b3 Merge branch 'jedwards4b-more_mira_port' 1df3edd add path search for pylint 89c084f change CAM55 to CAM60 03520b1 merge to trunk 367d47f Merge branch 'jgfouca/add_code_checking' (PR #267) de33082 Fix a couple remaining pylint issues 1516c3b Made code_checker parallel, fix test name so it actually runs e46d0f4 Merge branch 'testreporter' 0c091cc remove whitespace 8a25823 Merge branch 'testreporter_update' of https://github.com/fischer-ncar/cime into testreporter 54e3916 Fix remaining test failures e777c55 Update testreporter to handle the new ESMCI TestStatus files. Put back ETEST compset needed for testing. 300cdce Merge pull request #268 from jedwards4b/pbs_fixes 84ebf2d remove commented code 037d253 get st_archive working on mira aa193c7 need compiler attribute in get_default_mpilib 30b5fcf remove colon from fixes line 42ad9e0 add SAVE_TIMING_DIR for blues b36e2cd Bug fix 380f8d4 fix regex match for blues c5d157e rework CIMEROOT capture in templates acb6008 sta working now 77b4ae1 more updates for mira b90daa8 changed so that the expression is evald a0c984d update acme side 8a5ef7f correct batch directives bfac10e get mira working again 706c28e Merge branch 'master' of https://github.com/ESMCI/cime 9a4db4d Merge pull request #261 from jedwards4b/nck_fix 4fff865 All scripts passing 553bba5 fixes for pbs systems 479ddea Fix missing import, dunno how this made it past testing 557cff3 Merge branch 'master' of https://github.com/ESMCI/cime ed3a74d Merge branch 'jgfouca/python3_better_error' merge to master fd7dd06 fix merge conflict c233f96 Add code checker test. Remove refactor disablings from scripts_regression_tests.py c23bb0b Progress. Tool added. build.py 100% ca97ca9 fix user mod support for multi instance 17fe594 got nck build right d9b7302 correct problems in nck test e681111 better fix for nck test issues 1abee3e if ntasks for a comp is 1 the ninst should not be > 1 8b6d137 Merge branch 'jgfouca/fix_acme_postbuild' (PR #259) fb3aeea Fix unit tests. Unsafe defaults were being used 9239984 Suffix all files with lid d4c503b Minor improvement to error message f70029d Python3 users should get a better error now d3de6ae Trying to get reasonable error when using python 3 e3dcd0b Minor fix bbf9ba7 Merge pull request #260 from mvertens/remove_esmf_interface 302c66c Port performance archiving scripts to python. 766f974 Merge branch 'remove_esmf' into remove_esmf_interface 5e13ecd removed NOC test 42d19e7 fixes for pre-alpha cesm tests e989229 Removed all code related to using the ESMF interfaces in driver_cpl directory 83e7e0f removed esmf interfaces 8fa604a Merge branch 'jedwards4b-read_xml_fix' 41c71bf updated to PR #255 c10289e Merge pull request #252 from ESMCI/santos/case-context-manager 77e5a78 needed a flush in systems_tests_common 215987a Add `read_only` flag to `Case` constructor. d7ae79b Add read-only mode to `Case` objects. f84aebb Make `Case` a context manager. 0f298cf Merge branch 'master' of https://github.com/ESMCI/cime c52b65c Merge pull request #251 from jedwards4b/test_updates2 cd50e4f Merge remote-tracking branch 'mydev/test_updates2' 87b9b7d add sanity check f684eaf added new version of seq_diag_mct.F90 and backed out new config_grids.xml schema dea5064 move case update to preview_namelist d1ed95a fixes erp test e0dda70 working on erp 4c4a57f a pythonization of the original csh scripts f9c8252 fixed bugs in erp.py - there are still outstanding problems dabbdce new schema (version 2.0) for config_grids.xml 3f5eca9 Merge remote-tracking branch 'origin/master' into test_updates2 d55ba8f fixed problems with order dependence of compset attributes ad26223 reordered elements such that grid search could be done correctly 3874918 merged to cesmdev cime cime4.5.22 a26d382 added comments to seq.py a65a6f3 added file back in - needed for regression tests dbde0f0 xmlchange_cmds was not called correctly - this is now fixed ec6f656 fixes to get tests working and major cleanup of utils/perl a1cbd50 updates for numerous tests and deletion from Testing/TestCases e20a53b added pem and erp tests 8e424dc fixed removal of CME tests 8b32dc3 Merge pull request #471 from fischer-ncar/Replace_Bcompsets f3dc375 Fix broken compset names 9bea53a Merge isotope updates and corip1 module updates ceac486 Merge pull request #469 from fischer-ncar/corip1_update 26918a5 Update modules for corip1 60d2fa5 Merge isotope updates to corip1_update 81102e9 Merge pull request #463 from cacraigucar/geotrace_cime d9a31c0 Merge tag 'cime4.5.20' into geotrace_cime b08f005 Merge remote-tracking branch 'upstream/master' e8d85ba Merge pull request #462 from fischer-ncar/mpas_o 3c9357d Merge updated origin. 10bc4ac Update ChangeLog 9be64db Merge pull request #467 from fischer-ncar/ChangeLog 8d45e7e Update ChangeLog ae57982 Merge remote-tracking branch 'upstream/master' 27a22d1 Merge pull request #465 from fischer-ncar/ChangeLog baec797 Add new ocn ice coupler fields f17545b Merge pull request #459 from apcraig/marbl de371db Update to add WaveWatch support 467b8ea Merge pull request #464 from fischer-ncar/WaveWatch 15b3bea Update ChangeLog, fix typo in config_compset.xml 3767ae8 Removed allactive compsets that weren't being tested. 49b2b47 Updates for WaveWatch bc93250 Changes to config_files.xml and config_grids.xml to add mpas-o 43d5d0e Merge tag 'cime4.5.17' into geotrace_cime 9e2603e Merge tag 'cime4.5.14' into geotrace_cime 1837319 Merge tag 'cime4.5.10.1' into geotrace_cime 1c8ef5b Merge tag 'cime4.5.6' into geotrace_cime a9a5ac9 Merge tag 'cime4.4.9' into geotrace_cime 7e4143e Add bcphi, bcpho, flxdst ice to ocean coupling fields c4a9548 Remove ESMF tests and add ChangeLog bc5524e git reset --hard geotrace_cime_n13_cime4.4.8 and add aux_isotope test 9eae658 update esp_present bug for cime4.5.10.1 eba705a Merge tag 'geotrace_cime_n12_cime4.4.8' into geotrace_cime 43e18b8 [ 50 character, one line summary ] add Mariana's fix for CO2 for Isotopes dd34c44 Fix esp assignment in seq_rest_mod.F90. This will go into cesm1_5_beta6.1 a4be399 Merge tag 'cime4.4.8' into geotrace_cime 4410a89 Revert "Revert "Merge tag 'cime4.4.7' into geotrace_cime"" f76d977 Revert "Merge tag 'cime4.4.7' into geotrace_cime" c8f9137 Merge tag 'cime4.4.7' into geotrace_cime d5df824 Merge remote-tracking branch 'upstream/master' e598bea Merge tag 'cime4.2.3' into geotrace_cime cd5c2c9 Merge tag 'cime4.0.3' into geotrace_cime 502959b Merge tag 'cime3.0.7' into geotrace_cime a552bff Merge tag 'cime2.0.18-p1.1' into geotrace_cime 1fafc89 Merge tag 'cime2.0.0' into geotrace_cime 0f0ff2b Add -ntr_iso to CICE config for isotopes ea874e9 Put in correct CMakeLists.txt e96f3b2 Merge tag 'cime1.1.10' into geotrace_cime 3d72660 modified: driver_cpl/bld/namelist_files/namelist_definition_drv.xml new file: driver_cpl/driver/mrg_mod.F90 modified: driver_cpl/driver/prep_ice_mod.F90 modified: driver_cpl/driver/prep_ocn_mod.F90 modified: driver_cpl/driver/prep_rof_mod.F90 modified: driver_cpl/driver/seq_diag_mct.F90 modified: driver_cpl/driver/seq_flux_mct.F90 modified: driver_cpl/shr/seq_flds_mod.F90 modified: externals/pio/CMakeLists.txt modified: machines/config_pes.xml modified: scripts/Tools/config_compsets.xml modified: scripts/Tools/config_definition.xml modified: share/csm_share/shr/shr_const_mod.F90 modified: share/csm_share/shr/shr_flux_mod.F90 new file: share/csm_share/shr/water_isotopes.F90 new file: share/csm_share/shr/water_types.F90 - rest of files from geotracer svn branch 99e4eeb modified: driver_cpl/bld/build-namelist - added first file for geotrace branch 32b1b7d Merge tag 'cime2.0.17-p1.1' into cime2.0.18-p1 cebd23c patch to fix resubmit in cesm1_3_beta07 git-subtree-dir: cime git-subtree-split: 92a5d03
jgfouca
pushed a commit
that referenced
this pull request
Feb 27, 2018
Makes the processor layout for each component on Edison be a multiple of 24 for ne16 grid. Fixes #461 [BFB] * bishtgautam/scripts/ne16-edison-test-fix: Corrects processor layout on Edison for ne16 grid Conflicts: cime/machines-acme/config_pes.xml
yunpengshan2014
pushed a commit
that referenced
this pull request
Apr 2, 2024
Patch to turn off image comparison
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Makes the processor layout for each component on Edison be
a multiple of 24 for ne16 grid.
Fixes #461
[BFB]