Adding script to check RT logs and output#60
Conversation
|
@mrinalbiswas Thanks for preparing and adding this useful tool/script to check HAFS regression test results. Currently, we are almost ready to complete this PR #58. I was wondering if you could test this script with that branch, after that we will working on merge this PR. Thanks! |
|
@BinLiu-NOAA Sure. I am planning to test it on Orion once the runs are over. The runs on Orion are going super slow. @evankalina is planning to test it on Hera RT outputs. If they are all successful you can merge this PR. |
|
I have tested the hafs_rt_status.sh script on PR #58, and it worked without any changes. I liked using the script as it saves a lot of time checking the various rocoto XMLs, log files, etc. |
|
@mrinalbiswas I made some minor changes on top of your original version, and is trying to get back to your branch through this PR. Please feel free to take or discard the changes I made. Thanks! |
|
@mrinalbiswas Could you please sync your feature/hafs_rt_status branch: Thanks! |
|
@bin Liu - NOAA Affiliate ***@***.***> It should be synced with
develop now.
…On Mon, May 10, 2021 at 1:51 PM Bin Liu ***@***.***> wrote:
@mrinalbiswas <https://github.com/mrinalbiswas> Could you please sync
your feature/hafs_rt_status branch:
https://github.com/mrinalbiswas/HAFS/tree/feature/hafs_rt_status
with the latest HAFS develop branch?
https://github.com/hafs-community/HAFS/tree/develop
After that, I think we will be ready to merge this PR.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#60 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AE7WSBYF2V3XJRQJGRNQG7DTNA2N7ANCNFSM43NH4SIQ>
.
--
*My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.*
|
|
@mrinalbiswas, for whatever reason, this PR is not working properly to use the latest commit of your feature/hafs_rt_status branch. Could you please close this PR and recreate a new PR to merge your feature/hafs_rt_status branch into the hafs-community develop branch? Thanks! |
|
Closing this PR.. @BinLiu-NOAA reported an issue merging the PR. Will open a new one. |
Adding script to check the logs and output for HAFS application/workflow level regression tests. This PR addresses issue #60.
This pull request addresses issue #52. * Switch to use MSLET (instead of PRMSL) in the vortex tracker. * Update namelist.gettrk_tmp to use MSLET (from Jili).
Adding script to check the logs and output for HAFS application/workflow level regression tests. This PR addresses issue #60.
This pull request addresses issue hafs-community#52. * Switch to use MSLET (instead of PRMSL) in the vortex tracker. * Update namelist.gettrk_tmp to use MSLET (from Jili).
3 participants
Adding script to check the logs and output for HAFS application/workflow level regression tests.