-
Notifications
You must be signed in to change notification settings - Fork 93
Updated AMS models #1020
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
base: master
Are you sure you want to change the base?
Updated AMS models #1020
Conversation
jamelvin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, some clean up needed and some consistency across different AMS files needs to be added
da815d1 to
9d79763
Compare
|
On NVIDIA GPU: |
7b31841 to
cbba122
Compare
|
Does anyone know why the CI fails here? |
|
@marchdf the docker container we use to drive CI is busted. I put in a fix for this to |
|
@psakievich can we merge this? I am worried that a bunch of nalu-wind modernization is going to pass us by... |
|
@marchdf I would have merged it but the CI has been failing. I think it might be good to do a mini-hack-a-thon to update some of these PR's. I don't want this one to stay in purgatory any longer, but I also have quite a bit of changes to how fields are registered coming down the pipeline. I think it will make the code simpler. I am not planning to start that work until I get tooling updated a bit though. So I might suggest waiting until I get that started... |
Still left to do:
Co-authored-by: Jeremy Melvin [email protected]