Skip to content
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

ADCIRC v55 #20

Merged
merged 22 commits into from
May 14, 2021
Merged

ADCIRC v55 #20

merged 22 commits into from
May 14, 2021

Conversation

ghost
Copy link

@ghost ghost commented Mar 4, 2021

implemented @pvelissariou1 patch for ADCIRC v55 to the ADCIRC repository and also each submodule

@ghost ghost requested a review from saeed-moghimi-noaa March 4, 2021 21:32
@ghost ghost self-assigned this Mar 4, 2021
@ghost
Copy link
Author

ghost commented Mar 4, 2021

@ghost
Copy link
Author

ghost commented Mar 4, 2021

The build fails on Hera: #19

@ghost ghost linked an issue Mar 4, 2021 that may be closed by this pull request
@ghost
Copy link
Author

ghost commented Mar 5, 2021

both ADCIRC and NEMS now build successfully; now testing

@saeed-moghimi-noaa
Copy link
Collaborator

saeed-moghimi-noaa commented Mar 5, 2021 via email

@ghost ghost marked this pull request as ready for review March 8, 2021 21:04
@ghost ghost mentioned this pull request Mar 10, 2021
@ghost ghost force-pushed the feature/adcirc_v55 branch 2 times, most recently from 8505302 to 4a9fbae Compare March 16, 2021 15:27
@ghost ghost force-pushed the feature/adcirc_v55 branch from 4a9fbae to a856501 Compare March 16, 2021 15:29
@ghost
Copy link
Author

ghost commented Apr 8, 2021

these changes are built on top of @GM-Ling 's changes in #7

@ghost ghost mentioned this pull request Apr 8, 2021
@ghost ghost force-pushed the feature/adcirc_v55 branch from c4c4883 to fa8c331 Compare May 13, 2021 18:00
@ghost ghost requested review from saeed-moghimi-noaa and removed request for saeed-moghimi-noaa May 13, 2021 18:07
@saeed-moghimi-noaa saeed-moghimi-noaa merged commit 4b7dafe into master May 14, 2021
@saeed-moghimi-noaa
Copy link
Collaborator

@zacharyburnettNOAA
Hi Zach, I have merged the branch. Good luck. -Saeed

@ghost ghost deleted the feature/adcirc_v55 branch June 4, 2021 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

v55 build fails on Hera
3 participants