-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Regression tests on CircleCI and Parallel works #114
Comments
Takis, Did All the tests go through? -Saeed
…______________________________________________________
Saeed Moghimi, PhD
*NOS Storm Surge Modeling
<https://nauticalcharts.noaa.gov/learn/storm-surge-modeling.html> Team Lead*
Coastal Marine Modeling Branch, Coast Survey Development Laboratory, Office
of Coast Survey at NOAA National Ocean Service.
Address: 1315 East West Hwy, Room 6607, Silver Spring, Maryland 20910
Phone: (302) 827-3986
The contents of this message are mine personally and do not necessarily
reflect any position of NOAA.
On Tue, Jun 28, 2022 at 1:19 PM Panagiotis Velissariou < ***@***.***> wrote:
Assigned #114 <#114>
to @saeed-moghimi-noaa <https://github.com/saeed-moghimi-noaa>.
—
Reply to this email directly, view it on GitHub
<#114 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/APZULDY2MCX7DPFO2N4WK73VRMX2FANCNFSM52C5SK6Q>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
This issue follows the details in issue #113 |
On CircleCI it seems that for the free plan we can choose up to 4 cpu / 8 GB RAM |
@pvelissariou1 @SorooshMani-NOAA @saeed-moghimi-noaa
For CI:
For PW:
Seems like https://github.com/ufs-community/ufs-weather-model/blob/develop/.github/workflows/aux.yml are using EC2 with github action |
From meeting with Ali
|
The tests for PaHM go as expected. Working on the yaml file and token registration for codecov and ADCIRC. Minimum Core/CPU requirements for regression tests |
If the GitHub action tests are set properly (container + runner machine) do we need this?
The text was updated successfully, but these errors were encountered: