NOAA-EMC / TC_tracker

4 stars 6 forks source link

Migrate to Rocky8 installation of spack stack on Jet #6

Open InnocentSouopgui-NOAA opened 5 months ago

InnocentSouopgui-NOAA commented 5 months ago

Description

This change affects only Jet.

How has this been tested?

Cycled Global workflow experiments (48+ hours) on Jet at resolutions

InnocentSouopgui-NOAA commented 5 months ago

@KateFriedman-NOAA , I am tagging you for the TC tracker pull request.

InnocentSouopgui-NOAA commented 5 months ago

@KateFriedman-NOAA , it failed in another run, more check before I confirm.

InnocentSouopgui-NOAA commented 5 months ago

@KateFriedman-NOAA , it failed in another run, more check before I confirm.

All good now. I forgot to fix the detection of Jet at one location in a script. You can proceed with the review.

KateFriedman-NOAA commented 5 months ago

@JiayiPeng-NOAA FYI, @InnocentSouopgui-NOAA has done the work to update TC_tracker to build with spack-stack on Jet under the new Rocky8 OS. This PR may provide you and @HananehJafary-NOAA some guidance for similar updates on the other R&D machines (e.g. spack-stack and Rocky8/9).

Please review and approve if these changes look good to you. Thanks!

InnocentSouopgui-NOAA commented 5 months ago

@JiayiPeng-NOAA, Have you had the chance to look at this pull request?

HananehJafary-NOAA commented 5 months ago

@InnocentSouopgui-NOAA I'm testing it right now.

HananehJafary-NOAA commented 5 months ago

@InnocentSouopgui-NOAA @KateFriedman-NOAA @JiayiPeng-NOAA I have updated the TC_tracker code for both Orion and Jet. @InnocentSouopgui-NOAA, thank you for your update. I have compiled your code as well; it didn't run into any issues. However, I have cloned the same code I originally worked on Orion to keep it consistent across all the machines.

JiayiPeng-NOAA commented 5 months ago

Hi Hana, I guess you are referring to Hera, instead of Orion. Thanks, Jiayi

On Fri, Apr 26, 2024 at 9:47 PM Hana @.***> wrote:

@InnocentSouopgui-NOAA https://github.com/InnocentSouopgui-NOAA @KateFriedman-NOAA https://github.com/KateFriedman-NOAA @JiayiPeng-NOAA https://github.com/JiayiPeng-NOAA I have updated the TC_tracker code for both Orion and Jet. @InnocentSouopgui-NOAA https://github.com/InnocentSouopgui-NOAA, thank you for your update. I have compiled your code as well; it didn't run into any issues. However, I have cloned the same code I originally worked on Orion to keep it consistent across all the machines.

— Reply to this email directly, view it on GitHub https://github.com/NOAA-EMC/TC_tracker/pull/6#issuecomment-2080311149, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALRTVNJZLZTSUHRG36BRTNDY7L7SDAVCNFSM6AAAAABGPQYF56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBQGMYTCMJUHE . You are receiving this because you were mentioned.Message ID: @.***>

HananehJafary-NOAA commented 5 months ago

@JiayiPeng-NOAA Yes sorry about the confusion. It works on Hera and Jet.

DavidHuber-NOAA commented 5 months ago

@JiayiPeng-NOAA Is there anything else that needs to be done for this PR?

JiayiPeng-NOAA commented 5 months ago

Hi David, Hana will deliver a TC tracker package working for both Hera and Jet. Please help to test the new version after it is delivered. Thanks, Jiayi

On Mon, May 6, 2024 at 7:45 AM David Huber @.***> wrote:

@JiayiPeng-NOAA https://github.com/JiayiPeng-NOAA Is there anything else that needs to be done for this PR?

— Reply to this email directly, view it on GitHub https://github.com/NOAA-EMC/TC_tracker/pull/6#issuecomment-2095821661, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALRTVNK4JE2KVXQPIUQWKT3ZA5ULVAVCNFSM6AAAAABGPQYF56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJVHAZDCNRWGE . You are receiving this because you were mentioned.Message ID: @.***>