Closed BinLiu-NOAA closed 4 years ago
BinLiu-NOAA
commented
4 years ago I have synced all the hafs submodules with their latest develop or master branches in the hafs-community fork's feature/sync_submodule_20200324 (link). Trying to build and test this branch right now.
BinLiu-NOAA
commented
4 years ago Hi @ChunxiZhang-NOAA @JiliDong-NOAA and @AndrewHazelton,
My build on Hera failed with this kind of error:
INFO: Checking optional arguments in physics schemes ... INFO: Metadata table for model FV3 written to /scratch1/NCEPDEV/hwrf/save/Bin.Liu/hafs_202003/sorc/hafs_forecast.fd/FV3/ccpp/framework/doc/DevelopersGuide/CCPP_VARIABLES_FV3.tex INFO: Comparing metadata for requested and provided variables ... ERROR: Variable atmosphere_momentum_diffusivity requested by MODULE_satmedmfvdifq SCHEME_satmedmfvdifq SUBROUTINE_satmedmfvdifq_run & MODULE_hedmf_hafs SCHEME_hedmf_hafs SUBROUTINE_hedmf_hafs_run not provided by the model
@ChunxiZhang-NOAA could you please help to check out this hafs-community fork's feature/sync_submodule_20200324 (link) and give it try to see if I have missed something during the merge/sync of the submodules?
Thanks!
Bin
AndrewHazelton
commented
4 years ago Bin,
Hmmmm, I got that error at one point when the dkudiagnostic variable in GFS_typedefs.meta under hafs_forecast.fd/FV3/gfsphysics/GFS_layer/ didn't match with:
hafs_forecast.fd/FV3/ccpp/physics/physics/satmedmfvdifq.meta
But that was corrected in the version I submitted in the pull request.
Andy
On Wed, Mar 25, 2020 at 3:16 PM Bin Liu notifications@github.com wrote:
Hi @ChunxiZhang-NOAA https://github.com/ChunxiZhang-NOAA @JiliDong-NOAA https://github.com/JiliDong-NOAA and @AndrewHazelton https://github.com/AndrewHazelton,
My build on Hera failed with this kind of error:
INFO: Checking optional arguments in physics schemes ... INFO: Metadata table for model FV3 written to /scratch1/NCEPDEV/hwrf/save/Bin.Liu/hafs_202003/sorc/hafs_forecast.fd/FV3/ccpp/framework/doc/DevelopersGuide/CCPP_VARIABLES_FV3.tex INFO: Comparing metadata for requested and provided variables ... ERROR: Variable atmosphere_momentum_diffusivity requested by MODULE_satmedmfvdifq SCHEME_satmedmfvdifq SUBROUTINE_satmedmfvdifq_run & MODULE_hedmf_hafs SCHEME_hedmf_hafs SUBROUTINE_hedmf_hafs_run not provided by the model
@ChunxiZhang-NOAA https://github.com/ChunxiZhang-NOAA could you please help to check out this hafs-community fork's feature/sync_submodule_20200324 (link https://github.com/hafs-community/HAFS/tree/feature/sync_submodule_20200324) and give it try to see if I have missed something during the merge/sync of the submodules?
Thanks!
Bin
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NOAA-EMC/HAFS/issues/41#issuecomment-604034159, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMCUFFAWPG7J6GYDA2PUVLDRJJJ6ZANCNFSM4LTVECDA .
AndrewHazelton
commented
4 years ago Did part of my pull request for the GFS_layer part not make it through?
I'm looking at GFS_typedefs.meta and I do not see the "dkudiagnostic" variable I defined.
On Wed, Mar 25, 2020 at 3:32 PM Andrew Hazelton - NOAA Affiliate < andrew.hazelton@noaa.gov> wrote:
Bin,
Hmmmm, I got that error at one point when the dkudiagnostic variable in GFS_typedefs.meta under hafs_forecast.fd/FV3/gfsphysics/GFS_layer/ didn't match with:
hafs_forecast.fd/FV3/ccpp/physics/physics/satmedmfvdifq.meta
But that was corrected in the version I submitted in the pull request.
Andy
On Wed, Mar 25, 2020 at 3:16 PM Bin Liu notifications@github.com wrote:
Hi @ChunxiZhang-NOAA https://github.com/ChunxiZhang-NOAA @JiliDong-NOAA https://github.com/JiliDong-NOAA and @AndrewHazelton https://github.com/AndrewHazelton,
My build on Hera failed with this kind of error:
INFO: Checking optional arguments in physics schemes ... INFO: Metadata table for model FV3 written to /scratch1/NCEPDEV/hwrf/save/Bin.Liu/hafs_202003/sorc/hafs_forecast.fd/FV3/ccpp/framework/doc/DevelopersGuide/CCPP_VARIABLES_FV3.tex INFO: Comparing metadata for requested and provided variables ... ERROR: Variable atmosphere_momentum_diffusivity requested by MODULE_satmedmfvdifq SCHEME_satmedmfvdifq SUBROUTINE_satmedmfvdifq_run & MODULE_hedmf_hafs SCHEME_hedmf_hafs SUBROUTINE_hedmf_hafs_run not provided by the model
@ChunxiZhang-NOAA https://github.com/ChunxiZhang-NOAA could you please help to check out this hafs-community fork's feature/sync_submodule_20200324 (link https://github.com/hafs-community/HAFS/tree/feature/sync_submodule_20200324) and give it try to see if I have missed something during the merge/sync of the submodules?
Thanks!
Bin
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NOAA-EMC/HAFS/issues/41#issuecomment-604034159, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMCUFFAWPG7J6GYDA2PUVLDRJJJ6ZANCNFSM4LTVECDA .
BinLiu-NOAA
commented
4 years ago Hi Andy,
I could not see the dkudiagnostic in the GFS_typedefs.meta file either. Could you please point me to a branch, which has this in the GFS_typedefs.meta file?
I can try to compare the branches and see if I can pull over the changes.
Thanks!
Bin Liu IMSG at NOAA/NWS/NCEP/EMC 5830 University Research CT Cubicle 2026 College Park, MD 20740 Tel: (301)-683-3763 Email: Bin.Liu@noaa.gov
On Wed, Mar 25, 2020 at 3:44 PM AndrewHazelton notifications@github.com wrote:
Did part of my pull request for the GFS_layer part not make it through?
I'm looking at GFS_typedefs.meta and I do not see the "dkudiagnostic" variable I defined.
On Wed, Mar 25, 2020 at 3:32 PM Andrew Hazelton - NOAA Affiliate < andrew.hazelton@noaa.gov> wrote:
Bin,
Hmmmm, I got that error at one point when the dkudiagnostic variable in GFS_typedefs.meta under hafs_forecast.fd/FV3/gfsphysics/GFS_layer/ didn't match with:
hafs_forecast.fd/FV3/ccpp/physics/physics/satmedmfvdifq.meta
But that was corrected in the version I submitted in the pull request.
Andy
On Wed, Mar 25, 2020 at 3:16 PM Bin Liu notifications@github.com wrote:
Hi @ChunxiZhang-NOAA https://github.com/ChunxiZhang-NOAA @JiliDong-NOAA https://github.com/JiliDong-NOAA and @AndrewHazelton https://github.com/AndrewHazelton,
My build on Hera failed with this kind of error:
INFO: Checking optional arguments in physics schemes ... INFO: Metadata table for model FV3 written to
/scratch1/NCEPDEV/hwrf/save/Bin.Liu/hafs_202003/sorc/hafs_forecast.fd/FV3/ccpp/framework/doc/DevelopersGuide/CCPP_VARIABLES_FV3.tex INFO: Comparing metadata for requested and provided variables ... ERROR: Variable atmosphere_momentum_diffusivity requested by MODULE_satmedmfvdifq SCHEME_satmedmfvdifq SUBROUTINE_satmedmfvdifq_run & MODULE_hedmf_hafs SCHEME_hedmf_hafs SUBROUTINE_hedmf_hafs_run not provided by the model
@ChunxiZhang-NOAA https://github.com/ChunxiZhang-NOAA could you please help to check out this hafs-community fork's feature/sync_submodule_20200324 (link < https://github.com/hafs-community/HAFS/tree/feature/sync_submodule_20200324 ) and give it try to see if I have missed something during the merge/sync of the submodules?
Thanks!
Bin
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NOAA-EMC/HAFS/issues/41#issuecomment-604034159, or unsubscribe < https://github.com/notifications/unsubscribe-auth/AMCUFFAWPG7J6GYDA2PUVLDRJJJ6ZANCNFSM4LTVECDA
.
— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/NOAA-EMC/HAFS/issues/41#issuecomment-604047652, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE7VJNOGNCUEP2MMNM76DJDRJJNK3ANCNFSM4LTVECDA .
BinLiu-NOAA
commented
4 years ago Hi @AndrewHazelton,
I could not see the dkudiagnostic in the GFS_typedefs.meta file either. Could you please point me to a branch, which has this in the GFS_typedefs.meta file?
I can try to compare the branches and see if I can pull over the changes.
Thanks!
Bin
AndrewHazelton
commented
4 years ago Here, it's in this branch:
https://github.com/hafs-community/fv3atm/tree/feature/add_km/gfsphysics/GFS_layer
That should have the correct GFS_typedefs.meta
Let me know if there's a problem.
Andy
On Wed, Mar 25, 2020 at 3:57 PM Bin Liu notifications@github.com wrote:
Hi @AndrewHazelton https://github.com/AndrewHazelton,
I could not see the dkudiagnostic in the GFS_typedefs.meta file either. Could you please point me to a branch, which has this in the GFS_typedefs.meta file?
I can try to compare the branches and see if I can pull over the changes.
Thanks!
Bin
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NOAA-EMC/HAFS/issues/41#issuecomment-604053576, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMCUFFBT6LAKGP524TRUDKLRJJOYTANCNFSM4LTVECDA .
BinLiu-NOAA
commented
4 years ago Thanks @AndrewHazelton !
I have merged the changes you just committed from this feature/add_km branch. The build is working now. At least, it has passed the previous failure point. Will let you know if I encounter other issues.
Thanks!
Bin
Here, it's in this branch: https://github.com/hafs-community/fv3atm/tree/feature/add_km/gfsphysics/GFS_layer That should have the correct GFS_typedefs.meta Let me know if there's a problem. Andy … On Wed, Mar 25, 2020 at 3:57 PM Bin Liu @.***> wrote: Hi @AndrewHazelton https://github.com/AndrewHazelton, I could not see the dkudiagnostic in the GFS_typedefs.meta file either. Could you please point me to a branch, which has this in the GFS_typedefs.meta file? I can try to compare the branches and see if I can pull over the changes. Thanks! Bin — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#41 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMCUFFBT6LAKGP524TRUDKLRJJOYTANCNFSM4LTVECDA .
AndrewHazelton
commented
4 years ago Thanks - keep me posted!
On Wed, Mar 25, 2020 at 5:13 PM Bin Liu notifications@github.com wrote:
Thanks @AndrewHazelton https://github.com/AndrewHazelton !
I have merged the changes you just committed from this feature/add_km branch. The build is working now. At least, it has passed the previous failure point. Will let you know if I encounter other issues.
Thanks!
Bin
Here, it's in this branch: https://github.com/hafs-community/fv3atm/tree/feature/add_km/gfsphysics/GFS_layer That should have the correct GFStypedefs.meta Let me know if there's a problem. Andy … <#m-5861804629029690463_> On Wed, Mar 25, 2020 at 3:57 PM Bin Liu @.***> wrote: Hi @AndrewHazelton https://github.com/AndrewHazelton https://github.com/AndrewHazelton, I could not see the dkudiagnostic in the GFS_typedefs.meta file either. Could you please point me to a branch, which has this in the GFS_typedefs.meta file? I can try to compare the branches and see if I can pull over the changes. Thanks! Bin — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#41 (comment) https://github.com/NOAA-EMC/HAFS/issues/41#issuecomment-604053576>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMCUFFBT6LAKGP524TRUDKLRJJOYTANCNFSM4LTVECDA .
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NOAA-EMC/HAFS/issues/41#issuecomment-604090372, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMCUFFDTD3BGOJGFS5EOMQLRJJXVTANCNFSM4LTVECDA .
BinLiu-NOAA
commented
4 years ago Dear All,
@AndrewHazelton @JiliDong-NOAA @ChunxiZhang-NOAA @BijuThomas-NOAA @ZhanZhang-NOAA
The regression tests on Hera went through well. The develop branch is thus updated. And all the HAFS submodules in the develop branch have now been synced with their latest develop/master branches.
Issue closed.
Thanks!
Bin
Sync HAFS submodules with their latest develop or master branches.