NOAA-OWP / lstm

Other
12 stars 17 forks source link

Add ngen integration workflow to gh actions #41

Open madMatchstick opened 10 months ago

madMatchstick commented 10 months ago

Reminder to include a ngen integration workflow similar to other formulation repos (e.g. cfe). Currently, the existing workflow yaml just speaks to (1) standalone & (2) bmi unit testing scripts.

stcui007 commented 10 months ago

I am pretty preoccupied at the moment. Does anyone want to take a shot at it? It is fairly straightforward. Just copy over the ngen_integration.yml from one of the other submodules and make a few minor changes. I am ready to help if you needed.

On Fri, Jan 26, 2024 at 12:30 PM JessicaGarrett-NOAA < @.***> wrote:

Reminder to include a ngen integration workflow similar to other formulation repos (e.g. cfe https://github.com/NOAA-OWP/cfe/blob/master/.github/workflows/ngen_integration.yaml). Currently, the existing workflow yaml https://github.com/NOAA-OWP/lstm/blob/master/.github/workflows/test_unit_standalone.yml just speaks to (1) standalone & (2) bmi unit testing scripts.

— Reply to this email directly, view it on GitHub https://github.com/NOAA-OWP/lstm/issues/41, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACA4SRJNLGSFLDB7S5JD76TYQPY2ZAVCNFSM6AAAAABCMRVLJSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEYDENZQGA4DMMY . You are receiving this because you are subscribed to this thread.Message ID: @.***>

madMatchstick commented 10 months ago

I am pretty preoccupied at the moment. Does anyone want to take a shot at it? It is fairly straightforward. Just copy over the ngen_integration.yml from one of the other submodules and make a few minor changes. I am ready to help if you needed.

@stcui007 - I went ahead and assigned this to myself :) I'll take a stab at it next week and will reach out if any issues come about. -thanks!

stcui007 commented 10 months ago

@Jessica Garrett - NOAA Affiliate @.***> Thanks for taking this on.

On Fri, Jan 26, 2024 at 3:37 PM JessicaGarrett-NOAA < @.***> wrote:

I am pretty preoccupied at the moment. Does anyone want to take a shot at it? It is fairly straightforward. Just copy over the ngen_integration.yml from one of the other submodules and make a few minor changes. I am ready to help if you needed.

@stcui007 https://github.com/stcui007 - I went ahead and assigned this to myself :) I'll take a stab at it next week and will reach out if any issues come about. -thanks!

— Reply to this email directly, view it on GitHub https://github.com/NOAA-OWP/lstm/issues/41#issuecomment-1912729032, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACA4SRKFNRV2LBOTKQES7V3YQQOZ7AVCNFSM6AAAAABCMRVLJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJSG4ZDSMBTGI . You are receiving this because you were mentioned.Message ID: @.***>